Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions turtlebot3_fake_node/CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@
Changelog for package turtlebot3_fake
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

2.3.8 (2025-07-10)
------------------
* None

Choose a reason for hiding this comment

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

medium

The changelog entry * None is uninformative. Since the package version was bumped, there was a change. A changelog should briefly describe the changes in a new version to help users and maintainers track the evolution of the package. A more descriptive entry would be more helpful.

* Maintenance release.


2.3.6 (2025-06-19)
------------------
* None
Expand Down
2 changes: 1 addition & 1 deletion turtlebot3_fake_node/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
<package format="3">
<name>turtlebot3_fake_node</name>
<version>2.3.6</version>
<version>2.3.8</version>
<description>
Package for TurtleBot3 fake node. With this package, simple tests can be done without a robot.
You can do simple tests using this package on rviz without real robots.
Expand Down
4 changes: 4 additions & 0 deletions turtlebot3_gazebo/CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@
Changelog for package turtlebot3_gazebo
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

2.3.8 (2025-07-10)
------------------
* None

Choose a reason for hiding this comment

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

medium

The changelog entry * None is uninformative. Since the package version was bumped, there was a change. A changelog should briefly describe the changes in a new version to help users and maintainers track the evolution of the package. A more descriptive entry would be more helpful.

* Maintenance release.


2.3.6 (2025-06-19)
------------------
* None
Expand Down
2 changes: 1 addition & 1 deletion turtlebot3_gazebo/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
<package format="3">
<name>turtlebot3_gazebo</name>
<version>2.3.6</version>
<version>2.3.8</version>
<description>
Gazebo simulation package for the TurtleBot3
</description>
Expand Down
5 changes: 5 additions & 0 deletions turtlebot3_manipulation_gazebo/CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@
Changelog for package turtlebot3_manipulation_gazebo
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

2.3.8 (2025-07-10)
------------------
* Fixed the issue where the TurtleBot3 Manipulation Gazebo simulation was not working properly
* Contributors: Hyungyu Kim

2.3.6 (2025-06-19)
------------------
* None
Expand Down
2 changes: 1 addition & 1 deletion turtlebot3_manipulation_gazebo/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ find_package(ament_cmake REQUIRED)
# Install
################################################################################
install(
DIRECTORY config gazebo launch ros2_control rviz urdf worlds models
DIRECTORY config gazebo launch ros2_control rviz urdf worlds models meshes
DESTINATION share/${PROJECT_NAME}
)

Expand Down
1 change: 0 additions & 1 deletion turtlebot3_manipulation_gazebo/launch/gazebo.launch.py
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,6 @@ def generate_launch_description():
'world': world,
}.items(),
),

Node(
package='gazebo_ros',
executable='spawn_entity.py',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -97,12 +97,12 @@ def generate_launch_description():
arguments=[
'-topic', 'robot_description',
'-entity', 'turtlebot3_manipulation_system',
'-x', '0',
'-y', '0',
'-z', '0',
'-R', '0',
'-P', '0',
'-Y', '0'
'-x', '0.0',
'-y', '0.0',
'-z', '0.0',
'-R', '0.0',
'-P', '0.0',
'-Y', '0.0'
],
output='screen',
),
Expand Down
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
Binary file not shown.
3 changes: 2 additions & 1 deletion turtlebot3_manipulation_gazebo/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
<package format="3">
<name>turtlebot3_manipulation_gazebo</name>
<version>2.3.6</version>
<version>2.3.8</version>
<description>
Gazebo simulation package for the TurtleBot3 manipulation
</description>
Expand All @@ -23,5 +23,6 @@
<exec_depend>xacro</exec_depend>
<export>
<build_type>ament_cmake</build_type>
<gazebo_ros gazebo_model_path="${prefix}/..:${prefix}/models"/>

Choose a reason for hiding this comment

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

high

Using ${prefix}/.. to add the parent install directory to GAZEBO_MODEL_PATH is very broad and can have unintended side effects. It makes Gazebo search the entire install space for models, which can lead to slow startup times and model name collisions if other packages are installed.

A more robust and maintainable approach, following ROS best practices, is for each package that provides Gazebo models to export its own model path. For example, if this package depends on models from turtlebot3_gazebo, then turtlebot3_gazebo/package.xml should contain:

<export>
  <gazebo_ros gazebo_model_path="${prefix}/models"/>
</export>

This would correctly add the specific model path to the environment.

The current change is a workaround that hides a missing export in a dependency. I recommend removing the overly broad path to encourage fixing the root cause in the dependency package.

    <gazebo_ros gazebo_model_path="${prefix}/models"/>

</export>
</package>
5 changes: 5 additions & 0 deletions turtlebot3_simulations/CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,11 @@
Changelog for package turtlebot3_simulations
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

2.3.8 (2025-07-10)
------------------
* Fixed the issue where the TurtleBot3 Manipulation Gazebo simulation was not working properly
* Contributors: Hyungyu Kim

2.3.6 (2025-06-19)
------------------
* Included the TurtleBot3 Manipulation Gazebo simulation package in the metapackage's package.xml and updated the CI configuration accordingly
Expand Down
2 changes: 1 addition & 1 deletion turtlebot3_simulations/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
<package format="3">
<name>turtlebot3_simulations</name>
<version>2.3.6</version>
<version>2.3.8</version>
<description>
ROS 2 packages for TurtleBot3 simulations
</description>
Expand Down
Loading