gazebosim / gz-sim

Open source robotics simulator. The latest version of Gazebo.
https://gazebosim.org
Apache License 2.0
621 stars 251 forks source link

Update sensors with pending trigger immediately in Sensors system #2408

Closed shameekganguly closed 1 month ago

shameekganguly commented 1 month ago

🦟 Bug fix

Fixes https://github.com/gazebosim/gz-sensors/issues/426

Summary

Update Sensors system to ignore update rate for sensors that have pending triggers and update them immediately. Depends on upstream https://github.com/gazebosim/gz-sensors/pull/431.

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.

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 85.71429% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 65.92%. Comparing base (922eea7) to head (d8d351d). Report is 168 commits behind head on gz-sim8.

Files Patch % Lines
src/systems/sensors/Sensors.cc 85.71% 2 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## gz-sim8 #2408 +/- ## =========================================== + Coverage 64.88% 65.92% +1.03% =========================================== Files 360 327 -33 Lines 29448 31328 +1880 =========================================== + Hits 19108 20653 +1545 - Misses 10340 10675 +335 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.