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

Fix RgbdSensor_nws_ros2 camera data and distortion type handling #77

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kaidegast
Copy link
Contributor

  1. Optimized camera data handling: Data is now retrieved only during the first iteration and stored in a class variable, avoiding repeated requests in every cycle.
  2. Distortion management: Support for the none distortion type has been added, eliminating errors and reducing the number of log messages generated by yarprobotinterface.

Optimized camera data handling: Data is now retrieved only during the first iteration and stored in a class variable, avoiding repeated requests in every cycle.
Distortion management: Support for the none distortion type has been added, eliminating errors and reducing the number of log messages generated by yarprobotinterface.
@kaidegast kaidegast marked this pull request as draft November 29, 2024 15:40
@kaidegast kaidegast closed this Dec 2, 2024
@kaidegast kaidegast reopened this Dec 2, 2024
Copy link

sonarqubecloud bot commented Dec 2, 2024

@traversaro
Copy link
Member

Thanks a lot @kaidegast ! This may be quite useful also for other robots using rgbdSensor_nws_ros2 on embedded boards. The valgrind test failure is present also on master and tracked in #66, if you do not want to do other changes probably you can mark the PR ready for review? Thanks!

@kaidegast kaidegast marked this pull request as ready for review December 3, 2024 08:48
@kaidegast
Copy link
Contributor Author

Great! Then i can say all the necessary changes have been applied. I have marked the PR as ready for review.

@traversaro

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

Successfully merging this pull request may close these issues.

2 participants