Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature-Request] expose sharpness setting in the ROS driver #614

Open
abaeyens-imod opened this issue Oct 24, 2024 · 3 comments
Open

[Feature-Request] expose sharpness setting in the ROS driver #614

abaeyens-imod opened this issue Oct 24, 2024 · 3 comments
Labels
enhancement New feature or request

Comments

@abaeyens-imod
Copy link

why

When currently using the Luxonis cameras with the ROS driver, the default sharpness is set quite high and not adjustable. We would prefer that it can be set lower to avoid the current oversharpening artefacts and the amplified image noise.

what

  • A new parameter sharpness that adjusts the sharpening.
  • This parameter doesn't have to be dynamic.
  • We would appreciate seeing this also for Noetic.

how

At first sight: this setting is already available in the C++ API, so it's mostly a matter of exposing it in the ROS driver.

also

If possible, also expose the parameters luma_denoise and chroma_denoise.

@abaeyens-imod abaeyens-imod added the enhancement New feature or request label Oct 24, 2024
@Serafadam
Copy link
Collaborator

Hi, please check this branch to see if this implementation works for your case. You can find the new parameters here

@abaeyens-imod
Copy link
Author

@Serafadam just tested this together with #613, it works great and makes the Luxonis cameras considerably more usable for VSLAM. Thank you for addressing this swiftly. Two nits:

  • i_set_chroma_denoise works well for the RGB camera, but I'm not sure what it's supposed to do for the mono cameras.
  • the new parameters don't show up in rqt_reconfigure yet, I assume related to that they aren't yet listed in depthai_ros_driver/cfg/parameters.cfg.

@Serafadam
Copy link
Collaborator

Hi, please check the newest version, params have been changed to runtime. Regarding chroma denoise, I think it applies mostly in the cases where stereo pair is color, like OAK-D SR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants