Paul Bovbel

Perhaps the standard behaviour could be changed to leave the frame as is, unless a target frame is specified? Seems like some people have trouble using bag_to_pcd if tf data...

+1 A reasonable usage message would do the trick.

Looks like it, yes.

No preference. Never used before, so if you want to do one as an example, I can help implement on the rest.

Could probably make the error message more specific by either catching BagUnindexedException first, or printing the contents of the error message.

Thanks, you're right, there is no explicit `tf2` support. If you find time to fix this, a PR would be welcome.

Thanks! Yeah, when the time comes to release this, I'll figure out how to get vrpn included in foxy. I think it's actually gotten easier with colcon's direct support for...

No plans, but PRs are very welcome! I don't have a VRPN system to test with at the moment.

For sure, please do make a PR, I've spun up a foxy-devel branch.

This is great, thanks. I'll circle back around to review this as soon as I can. Merging into foxy is fine, we can re-arrange branches once this is released anyways.