gazebosim / gazebosim/sdformat

Deprecate `//sensor/camera/pose`

Open
#710 6 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C++
Stars
216
Forks
125
Avg merge
1d 14h
Merged PRs (30d)
14

Description

A camera is the only sensor that allows specifying a pose element on top of `//sensor/pose`. I wonder if there is any utility in this or if it could be depreciated.

I can see how this might be relevant to the idea of specifying that a camera is attached at some position, and origin/center of the lense model has some offset to that. However, I think this case can easily be covered by setting the position of the `//sensor` to that position and introducing a fixed link from the sensor mount to the lense origin. This is likely a more accurate description anyway since the mount point is connected to a rigid body (the camera casing) which itself is attached to the lense system.

@azeey Would I have to write a dedicated proposal for this, or should this live here?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.