Adding tags to Bags about bag-database HOT 11 CLOSED

Everything is designed to build with Java 1.8. What errors are you encountering?

from bag-database.

For what it's worth, it should work that way. I've been busy lately and haven't had a chance to test it, but I'll see if I can repeat that sometime soon...

from bag-database.

Well, the Readme says:

The messages on the topic should be newline-separated tags that are made of colon-separated key/value pairs; for example:
name: John Doe
email: [email protected]

How I understood this is that name: John Doe is the key and email: [email protected] is the value, so what you @joelsa want to achieve should follow a name\nJohn Doe syntax, I guess?

However, for me it doesn't work either this way while specifying METADATA_TOPICS="/annotation" with the /annotation topic existing as std_msgs/String and containing some messages.

from bag-database.

That's certainly one way to read it @cheffe112, however by looking at the code I am quite sure that the pairs should be of the format tagName + ": " + value

from bag-database.

I have the same problem and I would like to help you to fix it. To do that, I need information about what is the Java version that should be used to build the solution. I am trying to do it with version 1.8.0_151, but it is not okay unfortunately.

from bag-database.

Same behaviour that was mentioned in this bug, the tags are not get parsed, even the content of the bag file is correct. I am looking into the problem, right now I am debugging the code and hopefully will find the problem.

Regarding to the build, nothing. It was a problem with my machine - so user error. :)

from bag-database.

I have found the problem, the array that meant to contain the metadata topics is not set anywhere, therefore stays empty, therefore skips the tag reading. I have manually set the topic to the default one by hardcoding it - just to prove my idea - and it worked, so the parsing and persisting functionality works for the tags.

Shall I fix it?

from bag-database.

Sure, please submit a pull request.

from bag-database.

Alright, I will do in the next few days.

from bag-database.

Well, it took a long time to figure out, but this should finally be fixed.

from bag-database.

Something is still not okay, I think the config file is not generated by the script that you have just added but it exists beforehand - at least it looks like this with the latest version of development branch.

from bag-database.