gazebosim / gz-sim

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

Update camera sensors to use the `//sensor/camera/pose` sdf element. #2433

Closed iche033 closed 3 weeks ago

iche033 commented 3 weeks ago

🦟 Bug fix

Summary

Follow-up on https://github.com/gazebosim/gz-sim/pull/2425#issuecomment-2148165722. This is a behavior change so targeting main. The camera sensors will now take into account the //sensor/camera/pose sdf element. I also added an entry to Migration guide.

Checklist

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.

azeey commented 3 weeks ago

The //sensor/camera/pose element seems redundant with //sensor/pose. If we don't have a good reason to keep it, maybe it can be deprecated and removed from the SDFormat spec.

iche033 commented 3 weeks ago

I'm in favor of deprecating and then removing the tag. Closing this PR