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

Integrate mobile robots by interfacing with airo-tulip #147

Merged
merged 30 commits into from
Oct 21, 2024
Merged

Conversation

m-decoster
Copy link
Contributor

@m-decoster m-decoster commented Oct 7, 2024

Describe your changes

This branch integrates with airo-tulip - currently at version 0.1.0 - to control our KELO mobile platform. The contributions are located in airo-robots, and are fairly limited in scope, with most of the implementation being low-level code that is implemented in the airo-tulip repository. The interface in airo-robots simply communicates with the KELO platform over TCP (technically, via 0MQ), which is abstracted away by the KELORobile class.

The contributions follow the existing class structure of airo-robots, introducing a MobileRobot interface, which is implemented by the KELORobile class. The KELORobile class also adds some methods that are not present in the interface, because they are specific to the KELO robot.

As the airo-tulip repository is going to undergo further changes in the near future, this interface in AIRO-mono is also subject to changes. At this time, no breaking changes are expected, but also not ruled out.

Checklist

  • Have you modified the changelog?
  • If you made changes to the hardware interfaces, have you tested them using the manual tests?

@m-decoster m-decoster requested a review from tlpss October 7, 2024 13:18
@m-decoster m-decoster merged commit 86da798 into main Oct 21, 2024
25 checks passed
@m-decoster m-decoster deleted the tulip branch October 21, 2024 12:49
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.

3 participants