larics / timesync

A class for synchronizing sensor readings to the system clock
Apache License 2.0
11 stars 6 forks source link

Move debug_info topic up one level #7

Open dmiklic opened 7 years ago

dmiklic commented 7 years ago

Currently debug_info is published into the (parent) node's private workspace. If the whole node is pushed into a namespace (e.g. from a launch file), there will be unnecessary namespace duplication. The rationale for this design is that debug_info from several nodes using timestamp synchronizer would get published into the same topic. However, if we add a "sensor id" to the TimesyncDebug message, publishing everything to one topic will actually be desired behavior. In short, two changs are necessary:

@ojura we might still need to discuss a bit if this is a good idea or not