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

Add CycloneDDS to supported middlewares #95

Open
civerachb-cpr opened this issue Nov 25, 2024 · 0 comments
Open

Add CycloneDDS to supported middlewares #95

civerachb-cpr opened this issue Nov 25, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@civerachb-cpr
Copy link
Contributor

Originally proposed in #85

** Feature Summary **
Add CycloneDDS to supported RMW implementations in clearpath_config/common/types/rmw_implementation.py. This would allow users to select what middleware they want to use.

** Pros **
Enables more flexibility. Under some scenarios, CycloneDDS may be preferable to FastDDS.

** Cons **
MCU firmware on Clearpath's physical platforms use MicroROS. At present MicroROS does not operate well with CycloneDDS

Other notes
Simulation should be able to support CycloneDDS just fine, as should any physical platforms that do not use MicroROS (e.g. Husky A200).

We could leverage the new exception system to validate that the selected middleware is compatible with the selected physical platform, though that would be a non-trivial set of changes to the generators. We'd also need to be able to specify if the system is being used for a physical robot vs a simulation. It's probably doable, but right now it's a question of complexity to implement vs the potential benefits of CycloneDDS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant