ros2_documentation icon indicating copy to clipboard operation
ros2_documentation copied to clipboard

Add ROS 2 CLI module to Fast DDS Discovery Server Tutorial

Open jparisu opened this issue 4 years ago • 9 comments

Update in Fast DDS Discovery Server Tutorial for ROS 2.

New paragraph: ROS 2 Introspection. It explains how to use ROS 2 Introspection with ROS 2 CLI and ROS 2 Daemon using the Fast DDS Discovery Server.

Signed-off-by: jparisu [email protected]

jparisu avatar Jan 19 '21 14:01 jparisu

Hi @clalancette , could you please take a look at this tutorial extension for Fast DDS Discovery Server in ROS 2 Foxy? This came from an issue in fastrtps_rmw https://github.com/ros2/rmw_fastrtps/issues/499 and it extends the Discovery Server explanation.

jparisu avatar Jan 22 '21 06:01 jparisu

@clalancette, @hidmic Thank you for your revisions. Please @hidmic , check if the new updates in the changes you requested please you, and close them if so.

jparisu avatar Feb 01 '21 15:02 jparisu

This Pull Request has been marked as draft till the new features of Fast-DDS Discovery Server has been merged in Fast-DDS library and the documentation has been updated. https://github.com/eProsima/Fast-DDS/pull/1763

jparisu avatar Feb 17 '21 08:02 jparisu

This pull request has been mentioned on ROS Discourse. There might be relevant details there:

https://discourse.ros.org/t/discovery-problems-with-the-discovery-server/19900/3

ros-discourse avatar Apr 14 '21 00:04 ros-discourse

@jparisu is this ready to be re-reviewed?

hidmic avatar Jun 18 '21 19:06 hidmic

Sorry for the delay, we have been very busy lately. It is not ready, as we have implemented a new feature just for this specific propose that we would like it to appear in the tutorial. I will add it and set it as ready ASAP.

jparisu avatar Jun 22 '21 05:06 jparisu

@jparisu This has been sitting for quite a while, is this close to being ready, or can we close it out?

clalancette avatar Sep 28 '22 19:09 clalancette

@jparisu This has been sitting for quite a while, is this close to being ready, or can we close it out?

Let me add this to the list for next week!

EduPonz avatar Sep 29 '22 06:09 EduPonz

Let me add this to the list for next week!

Thanks, appreciated.

clalancette avatar Sep 29 '22 11:09 clalancette

I'm going to close this PR for now, as it has been in draft for a long time and still seems to need some work to complete. Feel free to reopen it once it is ready to be reviewed.

clalancette avatar Oct 21 '22 15:10 clalancette