Bug Fixed for the fish eye config

[wang59695487]

IsaacLab/source/extensions/omni.isaac.lab/test/sim/test_spawn_sensors.py

Line 78 in 9cc298e

# FIXME: This throws a warning. Check with Replicator team if this is expected/known.

The issue originates from the project type in FisheyeCameraCfg(https://github.com/isaac-sim/IsaacLab/blob/main/source/extensions/omni.isaac.lab/omni/isaac/lab/sim/spawners/sensors/sensors_cfg.py#L171). To properly use the API in Isaac Sim Rep, it should be "fisheyePolynomial" rather than variations like "fisheye_polynomial".

• fisheye_orthographic -> fisheyeOrthographic
• fisheye_equidistant -> fisheyeEquidistant
• fisheye_equisolid -> fisheyeEquisolid
• fisheye_polynomial -> fisheyePolynomial
• fisheye_spherical -> fisheyeSpherical

[command-z-z]

You are right! I'm curious why such a big bug has not been fixed. I will raise a PR

[command-z-z]

IsaacLab/source/extensions/omni.isaac.lab/test/sim/test_spawn_sensors.py

Line 78 in 9cc298e

# FIXME: This throws a warning. Check with Replicator team if this is expected/known.

The issue originates from the project type in FisheyeCameraCfg(https://github.com/isaac-sim/IsaacLab/blob/main/source/extensions/omni.isaac.lab/omni/isaac/lab/sim/spawners/sensors/sensors_cfg.py#L171). To properly use the API in Isaac Sim Rep, it should be "fisheyePolynomial" rather than variations like "fisheye_polynomial".

• fisheye_orthographic -> fisheyeOrthographic
• fisheye_equidistant -> fisheyeEquidistant
• fisheye_equisolid -> fisheyeEquisolid
• fisheye_polynomial -> fisheyePolynomial
• fisheye_spherical -> fisheyeSpherical

hi, I would like to know how you know the correct camera projection_type parameters. I checked the informations on Replicator camera website. Only finding the following type, Acttually, fisheye_polynomial isn't the correct parameter.
.

[command-z-z]

I found that the projection_type only supports the following six parameters in Isaac Sim GUI.

[RandomOakForest]

Thank you for submitting this. Will close this issue as there is a PR for this now. Thanks.