isaac-sim / IsaacLab

Unified framework for robot learning built on NVIDIA Isaac Sim
https://isaac-sim.github.io/IsaacLab
Other
2.34k stars 969 forks source link

[Proposal] Add camera local frame setter and both local and global frame getter #1034

Open lhy0807 opened 2 months ago

lhy0807 commented 2 months ago

Proposal

The current implementation of the Camera sensor only exposes the global pose setter. However, pose getter and local pose setter is also needed for my project needs.

Motivation

I need to get the pose of the camera, however this getter function has not been implemented in the current Camera sensor interface. Setting the camera local pose is also needed.

Alternatives

I could get the pose using self._view. However, I think a more proper way is to expose this function in the Camera class.

Additional context

N/A

Checklist

Acceptance Criteria

Add the criteria for which this task is considered done. If not known at issue creation time, you can add this once the issue is assigned.

mpgussert commented 2 months ago

Hello @lhy0807 !

On a cursory examination of the code, I also wasn't able to find a way to get this information without directly accessing self._view. We already use this to set_world_pose of the camera, so we should probably have an equivalent get_world_pose in the same way. Local and other non global frame transformations would then be handled in the usual way. Good suggestion!