NVIDIA-ISAAC-ROS / isaac_ros_visual_slam

Visual SLAM/odometry package based on NVIDIA-accelerated cuVSLAM
https://developer.nvidia.com/isaac-ros-gems
Apache License 2.0
832 stars 130 forks source link

does it support rational_polynomial calibration from camera_info or config file? #10

Closed FPSychotic closed 2 years ago

FPSychotic commented 2 years ago

i,m no able to make work a stereo camera, I think it is due to its rational_polynomial calibration , can be supported in a easy way? as config file?

hemalshahNV commented 2 years ago

We do not yet support rational polynomial distortion model in Elbrus visual odometry. You can work around this by rectifying the image using isaac_ros_image_proc first and then passing in the undistorted image with pinhole model into Elbrus. Adding rational polynomial distortion model support is being prioritized for a future release.

FPSychotic commented 2 years ago

Thanks, that workaround sounds nice to learn a little about, as I have no experience with calibrations and so on. So time to learn a little.

Thanks!

On Fri, 17 Dec 2021, 18:55 Hemal Shah, @.***> wrote:

We do not yet support rational polynomial distortion model in Elbrus visual odometry. You can work around this by rectifying the image using isaac_ros_image_proc first and then passing in the undistorted image with pinhole model into Elbrus. Adding rational polynomial distortion model support is being prioritized for a future release.

— Reply to this email directly, view it on GitHub https://github.com/NVIDIA-ISAAC-ROS/isaac_ros_visual_odometry/issues/10#issuecomment-996950508, or unsubscribe https://github.com/notifications/unsubscribe-auth/AITQOVEJKSEXKNG67HCWWDLUROBTJANCNFSM5KJPWXRQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are receiving this because you authored the thread.Message ID: @.*** com>

ciniminis1 commented 11 months ago

@hemalshahNV @FPSychotic Hi, has rational_polynomial calibration model been already released? Thank you.