-
Notifications
You must be signed in to change notification settings - Fork 512
Resolved an issue where tb3_manipulation simulation models were not loading #246
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,6 +2,10 @@ | |
Changelog for package turtlebot3_gazebo | ||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
||
2.3.8 (2025-07-10) | ||
------------------ | ||
* None | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The changelog entry
|
||
|
||
2.3.6 (2025-06-19) | ||
------------------ | ||
* None | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -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> | ||
|
@@ -23,5 +23,6 @@ | |
<exec_depend>xacro</exec_depend> | ||
<export> | ||
<build_type>ament_cmake</build_type> | ||
<gazebo_ros gazebo_model_path="${prefix}/..:${prefix}/models"/> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Using 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 <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.
|
||
</export> | ||
</package> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.