NVIDIA-ISAAC-ROS / isaac_ros_argus_camera

ROS 2 packages based on NVIDIA libArgus library for NVIDIA-accelerated CSI camera support.
https://developer.nvidia.com/isaac-ros-gems
Apache License 2.0
64 stars 12 forks source link

Missing documentation for calibration file #27

Open nakai-omer opened 1 year ago

nakai-omer commented 1 year ago

Running this package for a mono camera we are getting the following error:

Failed to get calibration data from Argus!

We have followed this guide for creating a calibration file, and after doing all the steps we have a camera_info.yaml file, but there is no explanation what we are supposed to do with this file. This packages seems to expect an ini file. So the bridge between the yaml and ini is missing in the docs.

nakai-omer commented 1 year ago

After al little more digging into the CameraInfoManager I saw that yaml files should also be supported. Nevertheless, trying to use the yaml file yielded the following error:

[component_container_mt-1] [INFO] [1689585165.778248505] [argus_mono]: camera calibration URL: package://isaac_ros_argus_camera/config/narrow_stereo.yaml
[component_container_mt-1] [WARN] [1689585165.782182253] [camera_calibration_parsers]: Exception parsing YAML camera calibration:
[component_container_mt-1] yaml-cpp: error at line 3, column 14: bad conversion
[component_container_mt-1] [ERROR] [1689585165.782326126] [camera_calibration_parsers]: Failed to parse camera calibration from file [/workspaces/isaac_ros-dev/install/isaac_ros_argus_camera/share/isaac_ros_argus_camera/config/narrow_stereo.yaml]

So I manually created an ini file from the yaml file, by structuring it like here. (UPDATE: the txt file created by the camera calibration pkg matches the ini file structure, so it can be used directly) Now the calibration file is being loaded as expected, but the following warning still exists:

amera.cpp@580: Failed to get calibration data from Argus!

I looked at the camera_info being published, and it matches the calibration info from the ini file.

swapnesh-wani-nvidia commented 1 year ago

Thank you for this issue. We will make the documentation clear.