Skip to content

Commit 71ee840

Browse files
Remove recommendation for Cyclone due to FastDDS issues (#799)
* Remove recommendation for Cyclone due to FastDDS issues * Remove cyclone config from Dockerfile
1 parent 7b88846 commit 71ee840

File tree

3 files changed

+0
-16
lines changed

3 files changed

+0
-16
lines changed

.docker/Dockerfile

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ RUN --mount=type=cache,target=/root/.ccache/ \
5353
. "/opt/ros/${ROS_DISTRO}/setup.sh" &&\
5454
. "install/setup.sh" &&\
5555
sudo apt update && rosdep install -r --from-paths src --ignore-src --rosdistro $ROS_DISTRO -y && \
56-
sudo apt install -y ros-${ROS_DISTRO}-rmw-cyclonedds-cpp && \
5756
colcon build \
5857
--cmake-args -DCMAKE_BUILD_TYPE=Debug -DBUILD_TESTING=OFF -DCMAKE_EXPORT_COMPILE_COMMANDS=ON \
5958
-DCMAKE_C_COMPILER_LAUNCHER=ccache -DCMAKE_CXX_COMPILER_LAUNCHER=ccache \
@@ -63,6 +62,5 @@ RUN --mount=type=cache,target=/root/.ccache/ \
6362
rm -rf /var/lib/apt/lists/* && \
6463
# Update /ros_entrypoint.sh to source our new workspace
6564
sed -i "s#/opt/ros/\$ROS_DISTRO/setup.bash#$ROS_UNDERLAY/install/setup.bash#g" /ros_entrypoint.sh && \
66-
echo "export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp" >> /root/.bashrc && \
6765
echo "source /opt/ros/${ROS_DISTRO}/setup.bash" >> /root/.bashrc && \
6866
echo "source /root/ws_moveit/install/setup.bash" >> /root/.bashrc

doc/tutorials/getting_started/getting_started.rst

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -108,19 +108,6 @@ Optional: add the previous command to your ``.bashrc``: ::
108108
not required and often skipped by advanced users who use more than one
109109
Colcon workspace at a time, but we recommend it for simplicity.
110110

111-
.. _cyclone_dds:
112-
113-
Switch to Cyclone DDS
114-
^^^^^^^^^^^^^^^^^^^^^
115-
116-
As of Sep 26, 2022, the default ROS 2 middleware (RMW) implementation has an issue.
117-
As a workaround, switch to Cyclone DDS.
118-
(Note: this makes all nodes started using this RMW incompatible with any other nodes not using Cyclone DDS.) ::
119-
120-
sudo apt install ros-rolling-rmw-cyclonedds-cpp
121-
# You may want to add this to ~/.bashrc to source it automatically
122-
export RMW_IMPLEMENTATION=rmw_cyclonedds_cpp
123-
124111
Next Step
125112
^^^^^^^^^
126113
Nice job!

doc/tutorials/quickstart_in_rviz/quickstart_in_rviz_tutorial.rst

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ Getting Started
99
---------------
1010
If you haven't already done so, make sure you've completed the steps in :doc:`Getting Started </doc/tutorials/getting_started/getting_started>` or our :doc:`Docker Guide </doc/how_to_guides/how_to_setup_docker_containers_in_ubuntu>`.
1111
If you followed the Docker Guide, also follow the :ref:`create_colcon_workspace` guide onwards to set up the tutorials.
12-
As of Sep 26, 2022, ensure you have :ref:`enabled Cyclone DDS <cyclone_dds>`.
1312

1413
Step 1: Launch the Demo and Configure the Plugin
1514
------------------------------------------------

0 commit comments

Comments
 (0)