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

Device connections #294

Merged
merged 3 commits into from
Jan 10, 2024
Merged

Device connections #294

merged 3 commits into from
Jan 10, 2024

Conversation

b1n-ch1kn
Copy link
Member

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Optimisation

[required] Description

Re-introduced DDS specifying for cyclone DDS, now with Domain ID.
Linked new SBG device with udev rule

[required] Documentation

Rule documented in configuration file QUTMS_Driverless/src/hardware/sensors/config/ellipse_D.yaml.
Docker documentation coming soon.

[optional] Usability concerns or breaking changes?

When ROS nodes are attemped to run without having their network adaptor linked (i.e. the ethernet connection) they will crash as messages cannot be transferred. Could look into dynamic DDS depending on connection availability.

@b1n-ch1kn b1n-ch1kn added the Refactoring Changing directories / workspaces or code tidy label Jan 10, 2024
@b1n-ch1kn b1n-ch1kn self-assigned this Jan 10, 2024
@b1n-ch1kn b1n-ch1kn merged commit 660d708 into master Jan 10, 2024
1 check passed
@b1n-ch1kn b1n-ch1kn deleted the ros-connections branch January 10, 2024 03:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Refactoring Changing directories / workspaces or code tidy
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant