Skip to content
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

Feature/ros2 #183

Merged
merged 123 commits into from
Oct 23, 2024
Merged

Feature/ros2 #183

merged 123 commits into from
Oct 23, 2024

Conversation

gabryelreyes
Copy link
Collaborator

No description provided.

gabryelreyes and others added 30 commits October 23, 2024 10:00
* Add webots subfolder with deployment
* Add reference to Webots ROS2 Supervisor
* some spelling improvements
* add correct deployment diagram for webots
* add new chapter to TOC of readme
* Added micro-ROS platoformio to DCS Turtle application with basic transport setup and rcl startup procedure and node spinning within loop; Added initial micro-ROS setup and usage documentation.

* Extracted microros configuration into own function

* Added basic file structure for MicroRosClient

* micro ros client extracted into own class

* Architecture of MicroRosClient

* Using subscription_with_context

* Implemented set config

* Removed subscription from Client source

* Correct destruction of node

* Try to create Subscriber and BaseSubscriber

* Implemented subscriber initialization

* Fixed documentation

* Renamed template

* Subscriber members are private

* Updated UML diagram

* Reverted config file

* Improved documentation and function names

* Test CI with fixed scripts

* fixed typo and corrected link to forked uros pio repo

* Fixed review findings

---------

Co-authored-by: hoeftjch <derpzillah@protonmail.com>
BlueAndi and others added 25 commits October 23, 2024 10:00
…the MQTT port, use the widely used port 8888 for the Micro-ROS agent. This makes it more clear.
…moved the fatal error infinite loops, which caused that the simulation was stuck.

#175
#176
This is now handled on application level by pinging the agent port.
Add c++ transport handler and  make the C-functions entry points
only wrappers for the new c++ ones.

This removes the need for a "handcrafted" this pointer inside the
C-methods for accessing Class elements.
- moved local functions the end of the file
- added "inline" to toThis() function
@gabryelreyes gabryelreyes marked this pull request as ready for review October 23, 2024 09:25
@gabryelreyes gabryelreyes merged commit 694ff9e into release/v1.0.x Oct 23, 2024
18 checks passed
@BlueAndi BlueAndi deleted the feature/ROS2 branch October 23, 2024 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants