Skip to content

Comments

Move MAVLink specific code from ArduPilot to MAVLink Vehicle#2125

Merged
patrickelectric merged 4 commits intobluerobotics:masterfrom
patrickelectric:move-mavlink-vehicle-2
Sep 12, 2025
Merged

Move MAVLink specific code from ArduPilot to MAVLink Vehicle#2125
patrickelectric merged 4 commits intobluerobotics:masterfrom
patrickelectric:move-mavlink-vehicle-2

Conversation

@patrickelectric
Copy link
Member

@patrickelectric patrickelectric commented Sep 12, 2025

This is a continuation for PX4 support

@@ -1,697 +1,24 @@
import { differenceInMilliseconds } from 'date-fns'
Copy link
Member

Choose a reason for hiding this comment

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

As the file is being completely moved, can we instead create a commit renaming the file and a separate one re-creating this one with the inheriting? This way we don't lose the direct commit history, which makes tracking changes much more difficult.

Copy link
Member Author

Choose a reason for hiding this comment

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

Check the latest change

Copy link
Member

Choose a reason for hiding this comment

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

Nice! Thanks!

- Rename `arduPilotGPSFixTable` → `gpsFixTable`
- Rename `ArduPilotParameterSetData` → `MAVLinkParameterSetData`
- Update `mainVehicle` store import for `MAVLINK_MESSAGE_INTERVALS_STORAGE_KEY`
- Update `mainVehicle` type to use `MAVLinkParameterSetData`

Signed-off-by: Patrick José Pereira <[email protected]>
… ardupilot original changes

Signed-off-by: Patrick José Pereira <[email protected]>
… keep track of orifinal changes

Signed-off-by: Patrick José Pereira <[email protected]>
…e generic mavlink vehicle

Signed-off-by: Patrick José Pereira <[email protected]>
@patrickelectric patrickelectric merged commit 59a94df into bluerobotics:master Sep 12, 2025
12 checks passed
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.

2 participants