Port sensor update rate fix

[iche033]

🦟 Bug fix

Summary

cherry-picked commit from #2863 to gazebo11

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

[peci1]

Thanks for the forward port. What about the missing mutex lock?

[iche033]

Thanks for the forward port. What about the missing mutex lock?

ah missed the note about mutex. Added. 82851d1

[traversaro]

@peci1 does the fix looks ready for merge to you? Thanks!

[peci1]

Yes, LGTM!

[traversaro]

Ok, I will leave some time for other people to comment, and then I will proceed to merge. Feel free to ping me if I forgot to merge.

[peci1]

So, is 2 weeks a long enough incubation period? :)

[traversaro]

So, is 2 weeks a long enough incubation period? :)

Even less, thanks for the ping!

[EmmanuelMess]

Sorry to comment on an old PR, but I must ask. Was the IMU provided by "GazeboRosImu" tested? Do other IMU plugins (custom made by third parties, like hector's IMU) need retrofiting?

[traversaro]

Sorry to comment on an old PR, but I must ask. Was the IMU provided by "GazeboRosImu" tested? Do other IMU plugins (custom made by third parties, like hector's IMU) need retrofiting?

If you have a doubt on GazeboRosImu, I suggest to open a (new) issue in https://github.com/ros-simulation/gazebo_ros_pkgs . For other IMUs, I suggest to open a new issue in this. However, it increases the chances that somebody replies if you provide more context to your questions, for example: are you experiencing problems with GazeboRosImu? Why do you think that third parties IMU plugin need to be retrofitted?