message_filters icon indicating copy to clipboard operation
message_filters copied to clipboard

rates_ vector out of channel order

Open andermi opened this issue 1 year ago • 2 comments

rates_ vector out of channel order when user-specified RateConfig vector provided

fixes https://github.com/ros2/message_filters/issues/110

andermi avatar Jan 23 '24 19:01 andermi

I'm honestly not sure that this is the correct fix. From what I can tell of the current code, it is actually possible (if non-intuitive) to configure the rates properly. You just need to call setRateConfigPerMessage of the correct size, and then all of the calls to add will reference the correct configuration as they are created. And then sort_indices will actually sort them by their expiry time.

However, I'm not 100% sure of all of that. What I think we really need is an example/test that shows the problem that we are trying to solve, and the way we would solve it with the current code. Then we can consider whether this proposal solves it better.

clalancette avatar Feb 12 '24 21:02 clalancette

Sounds good! I can try adding a test to replicate #110

andermi avatar Feb 12 '24 23:02 andermi