search

strands-project / strands_perception_people

long-term detection, tracking and recognition of people
96 stars 70 forks source link

Supporting the topological logging manager via launch files #160

Closed cdondrup closed 9 years ago

cdondrup commented 9 years ago

This adds the appropriate parameter to the launch files to use the topological logging manager or not.

Also now comparing for the empty string for the topic instead of None because there is no None.

cdondrup commented 9 years ago

Since we are now following the 4 eyes are better than two approach, @lucasb-eyer can you please have a look at this PR and #159 since it also touches on your packages.

lucasb-eyer commented 9 years ago

There is no None for me either then, could you please change that in the same PR too?

lucasb-eyer commented 9 years ago

Also, sorry, the second pair of eyes will be slow in the next three weeks because they're getting hit by the other project.

cdondrup commented 9 years ago

@lucasb-eyer thanks for checking. Will make the changes you asked for and commit before merging.