-
Notifications
You must be signed in to change notification settings - Fork 1k
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
fixed missed installation of the .yaml parameters file #4272
base: humble
Are you sure you want to change the base?
Conversation
In the section "2.2 Loading parameters from YAML file", it was missing the necessity of installing the .yaml file. I tested what is written in C++ Signed-off-by: AlessandroMelone <63403736+AlessandroMelone@users.noreply.github.com>
source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
Outdated
Show resolved
Hide resolved
source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
Outdated
Show resolved
Hide resolved
source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
Outdated
Show resolved
Hide resolved
…ge-Projects.rst added empty line before the code-block Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: AlessandroMelone <63403736+AlessandroMelone@users.noreply.github.com>
…ge-Projects.rst added empty line before the code-block Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: AlessandroMelone <63403736+AlessandroMelone@users.noreply.github.com>
removed empty line as suggested by ahcorde Signed-off-by: AlessandroMelone <63403736+AlessandroMelone@users.noreply.github.com>
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.
One more comment
source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
Outdated
Show resolved
Hide resolved
…ge-Projects.rst Co-authored-by: Alejandro Hernández Cordero <ahcorde@gmail.com> Signed-off-by: AlessandroMelone <63403736+AlessandroMelone@users.noreply.github.com>
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.
lgtm with code-block
category fix.
source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
Outdated
Show resolved
Hide resolved
source/Tutorials/Intermediate/Launch/Using-ROS2-Launch-For-Large-Projects.rst
Outdated
Show resolved
Hide resolved
…ge-Projects.rst Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com> Signed-off-by: AlessandroMelone <63403736+AlessandroMelone@users.noreply.github.com>
…ge-Projects.rst Co-authored-by: Tomoya Fujita <Tomoya.Fujita@sony.com> Signed-off-by: AlessandroMelone <63403736+AlessandroMelone@users.noreply.github.com>
Make sure that you have installed the ``turtlesim.yaml`` file. If you have a C++ package, write in the package ``CMakeLists.txt``: | ||
|
||
.. code-block:: cmake | ||
install(DIRECTORY | ||
config | ||
DESTINATION share/${PROJECT_NAME} | ||
) | ||
|
||
Instead, if you have a Python package, add in package ``setup.py`` file: | ||
|
||
.. code-block:: Python | ||
data_files=[ | ||
('share/ament_index/resource_index/packages', | ||
['resource/' + package_name]), | ||
('share/' + package_name, ['package.xml']), | ||
(os.path.join('share', package_name, 'launch'), glob('launch/*.launch.py')), | ||
(os.path.join('share', package_name, 'config'), glob('config/*.yaml')) | ||
], | ||
|
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.
This whole thing should go above the line If we now start...
above.
Further, the way we handle C++ vs. Python in other tutorials is through the use of group-tabs, and I think we should do the same here. You can see an example of this in action in
ros2_documentation/source/Tutorials/Demos/Logging-and-logger-configuration.rst
Lines 23 to 39 in d9bca49
.. tabs:: | |
.. group-tab:: C++ | |
.. code-block:: C++ | |
// printf style | |
RCLCPP_DEBUG(node->get_logger(), "My log message %d", 4); | |
// C++ stream style | |
RCLCPP_DEBUG_STREAM(node->get_logger(), "My log message " << 4); | |
.. group-tab:: Python | |
.. code-block:: python | |
node.get_logger().debug('My log message %d' % (4)) |
In the section "2.2 Loading parameters from YAML file", it was missing the necessity of installing the .yaml file. Missing that step, I was not generating the .yaml file in the /install folder, and therefore getting a runtime error.
I tested what is written in C++.
I was able to understand the solution to my problem thanks to this guide https://roboticsbackend.com/ros2-yaml-params/