Skip to content

Conversation

@veronikaslobo
Copy link

No description provided.

@thirteenlizards
Copy link
Contributor

Note: to be tested before merging

@thirteenlizards thirteenlizards requested review from Alb33rt and removed request for Alb33rt and andrew-kan October 21, 2025 01:54
@thirteenlizards
Copy link
Contributor

@Alb33rt @Rain1618 @nadimasmar please review + confirm the ros custom messages included here; ideally we do not modify this again so if there's anything you anticipate for the future let us know.

Copy link
Contributor

@thirteenlizards thirteenlizards left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Commit corresponding to mcgill-robotics/AUV-2025#752 "Update custom messages to sync with AUV-2026 message definitions"

Merge not currently needed, using this branch on its own for now with corresponding AUV-2025 branch.

Copy link
Member

@JPGC04 JPGC04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to decide whether to merge this pr.
This pr is the latest of what can work with AUV-2025 in ros1
However, it has not been tested extensively, just lightly.
If you want ros1 stuff to work in any way whatsoever i would recommens merginf this. But if you are 100% certain u are abandoning at all costs then i dont know id you should merge.
Take that decision but decide what to do with this pr so its not there forever.

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