Added CycloneDDS to the list of supported DDS

clearpathrobotics / clearpath_config

Clearpath Configuration YAML Parser

https://docs.clearpathrobotics.com/

BSD 3-Clause "New" or "Revised" License

9 stars 9 forks source link

Closed aalmrad closed 5 days ago

aalmrad commented 1 month ago

CycloneDDS provides more stable high frequency communication

tonybaltovski commented 3 weeks ago

Hi @aalmrad, thank you for the PR but we have not full qualified CycloneDDS for our platforms thus we can't accept this PR till we do.

aalmrad commented 3 weeks ago

Hello @tonybaltovski, I understand, thank you for your reply.

tonybaltovski commented 3 weeks ago

I'll leave this open as a way to collection thoughts on adding CycloneDDS support. For simulations, it should not matter, only on hardware.

civerachb-cpr commented 5 days ago

@tonybaltovski because open merge requests generate notifications for review, I'm going to close this. In its place I've created https://github.com/clearpathrobotics/clearpath_config/issues/95 to solicit feedback, write notes, etc... on this proposal.