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

Properly read camera config files in rs_launch.py #2572

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

Gilaadb
Copy link
Contributor

@Gilaadb Gilaadb commented Dec 7, 2022

Modified rs_launch.py to properly read parameters from yaml config file. It did not read them before.
Now we can set camera paramters that are not listen in rs_launch.py "configurable_parameters" list by specifying them in a yaml config file.

Example for such parameters could be (among many others): depth_module.enable_auto_exposure
depth_module.inter_cam_sync_mode

Modified rs_launch.py to properly read parameters from yaml config file.
It did not read them before.
Now we can set camera paramters that are not listen in rs_launch.py
"configurable_parameters" list by specifying them in a yaml config file.

Example for such parameters could be (among many others):
depth_module.enable_auto_exposure
depth_module.inter_cam_sync_mode
@Nir-Az
Copy link
Collaborator

Nir-Az commented Dec 7, 2022

Thanks for the PR Gilad, The changes looks very good.
Can you please elaborate how this was tested?
With config file + without? with dynamic changes on CLI?
ROS2 distro?

@Gilaadb
Copy link
Contributor Author

Gilaadb commented Dec 7, 2022

Hi Nir. I tested it on ROS Foxy, with and without a config file along with parameters that were already supported by the launch file (such as streams selection, resolution, IMU enablement etc...). Meaning the parameters from the yaml and from the launch arguments do merged to one list of parameters for the node.
It shouldn't change what is possible or not possible to change with dynamic changes on CLI - I tested it to some extent by tweaking some of the parameters via 'rqt' after a camera node start.

The main advantage you gain from applying this change, is to be able to set camera node parameters that you will see if you will do a "ros2 param dump /camera/camera" but are not included in the subset of parameters that are listed in the "configurable_parameters" variable in the rs_launch.py file.

@Nir-Az Nir-Az requested review from Nir-Az and removed request for SamerKhshiboun December 7, 2022 18:42
Copy link
Collaborator

@Nir-Az Nir-Az left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works like a charm :)

@Nir-Az Nir-Az merged commit 333f4fb into IntelRealSense:ros2-development Dec 7, 2022
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