Skip to content
This repository has been archived by the owner on Mar 23, 2024. It is now read-only.

Fix HelperROT.msg #34

Merged
merged 2 commits into from
Dec 18, 2023
Merged

Fix HelperROT.msg #34

merged 2 commits into from
Dec 18, 2023

Conversation

hhenry01
Copy link
Member

Description

  • HelperROT.msg had the wrong units datatype.

Verification

Resources

@hhenry01 hhenry01 added the bug Something isn't working label Dec 15, 2023
@hhenry01 hhenry01 requested a review from a team as a code owner December 15, 2023 06:27
@hhenry01
Copy link
Member Author

@SPDonaghy fyi

@hhenry01 hhenry01 enabled auto-merge (squash) December 15, 2023 07:46
@SPDonaghy
Copy link
Contributor

Just saw this, thanks Henry!

@jamenkaye
Copy link
Contributor

Interesting, nice catch Henry! Looks good to me. I assume this is specific to the AIS device we're using right?

@hhenry01
Copy link
Member Author

Interesting, nice catch Henry! Looks good to me. I assume this is specific to the AIS device we're using right?

This is part of the AIS standard: https://www.navcen.uscg.gov/ais-class-a-reports

@hhenry01 hhenry01 merged commit eb2392d into main Dec 18, 2023
12 checks passed
@hhenry01 hhenry01 deleted the user/hhenry01/fix-rot-message branch December 18, 2023 06:29
@patrick-5546 patrick-5546 mentioned this pull request Jan 1, 2024
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ubcsailbotsoftware ubcsailbotsoftware ubcsailbotsoftware approved these changes

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@hhenry01 @SPDonaghy @jamenkaye @ubcsailbotsoftware