Skip to content

Conversation

@avanmalleghem
Copy link

Related to issue : #12

I tested it on Jazzy. Here is an example of working commands :

ros2 run examples_rclrs_clap_demo clap_demo --count 15 --ros-args -r topic:=new_topic -- --name John

Signed-off-by: avanmalleghem <antoine.vanmalleghem@botronics.be>
Signed-off-by: avanmalleghem <antoine.vanmalleghem@botronics.be>
@esteve
Copy link
Contributor

esteve commented Nov 20, 2025

@avanmalleghem so cool, thanks! I can't speak for @mxgrey, but the way I understood the ticket was to integrate clap into some of the existing examples. While useful to show how clap can be used with rclrs, I'd prefer if it weren't a separate package that we'd have to maintain.

@avanmalleghem
Copy link
Author

@avanmalleghem so cool, thanks! I can't speak for @mxgrey, but the way I understood the ticket was to integrate clap into some of the existing examples. While useful to show how clap can be used with rclrs, I'd prefer if it weren't a separate package that we'd have to maintain.

Ok I let some time to @mxgrey so that we have his point of view aswell and I modify it if needed :) Thanks for your feedback !

Copy link
Contributor

@esteve esteve left a comment

Choose a reason for hiding this comment

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

@avanmalleghem what I meant is that given that the changes between this and the existing example are very minimal, I prefer not to maintain two packages that are almost identical. Could you integrate these changes into one of the existing examples? Thanks

@avanmalleghem
Copy link
Author

Fair point ! Will do it asap (max this weekend)

@esteve
Copy link
Contributor

esteve commented Nov 20, 2025

@avanmalleghem there's no rush 🙂 Thanks again for all the work!

@avanmalleghem
Copy link
Author

@esteve ok for you if I include clap into the rust_pubsub/src/simple_publisher.rs ?

@esteve
Copy link
Contributor

esteve commented Nov 20, 2025

@avanmalleghem I'd do it in https://github.com/ros2-rust/examples/tree/main/rclrs/minimal_pub_sub, as it's the most commonly used example. For example in the publisher and add a counter like you did in this PR. Could you make it so that if no arguments are passed, the example behaves like now? Thanks

Signed-off-by: avanmalleghem <antoine.vanmalleghem@botronics.be>
@avanmalleghem avanmalleghem requested a review from esteve November 20, 2025 17:45
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