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

image_publisher ignores filename and camera_info_url parameters #802

Closed
hpoirier opened this issue Feb 9, 2023 · 3 comments
Closed

image_publisher ignores filename and camera_info_url parameters #802

hpoirier opened this issue Feb 9, 2023 · 3 comments
Assignees

Comments

@hpoirier
Copy link

hpoirier commented Feb 9, 2023

When starting the image_publisher node, the filename parameter is ignored and the camera_info_url can only be set after the node is up using the ros2 set param command. Moreover filename is not declared in the image_publisher component but only in the image_publisher_node application which makes it impossible to use as a ComposableNode.
This requires some changes in the way parameters are handle when the node is created and in the on_set_parameters_callback.

@mikeferguson
Copy link
Member

Possibly same cause as #824?

@mikeferguson
Copy link
Member

The node part works with both parameter or command line arg:

ros2 run image_publisher image_publisher_node --ros-args -p filename:=test.png
ros2 run image_publisher image_publisher_node test.png

It does appear that filename is not properly defined when using the component alone, and also that the callback is not called on startup when using the component - should be fixed by #861

@mikeferguson mikeferguson self-assigned this Jan 19, 2024
@mikeferguson
Copy link
Member

PR merged

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

No branches or pull requests

2 participants