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

Add GPS message types, consolidate SBG messages, create common messag… #6

Merged
merged 2 commits into from
Nov 17, 2024

Conversation

NoahSprenger
Copy link
Contributor

This pull request includes significant changes to the src/command.rs, src/lib.rs, and src/sensor.rs files, as well as an update to the Cargo.toml file. The changes primarily focus on restructuring the message and sensor data handling, adding new message types, and removing redundant code.

Restructuring message and sensor data handling:

  • src/lib.rs: Introduced RadioMessage and CanMessage structs to differentiate between radio and CAN messages, and added RadioData and CanData enums to encapsulate different types of data. [1] [2]

Adding new message types:

  • src/sensor.rs: Added a comprehensive list of new GPS message types under the Gps enum, leveraging the ublox crate for various GPS data references.
  • src/sensor.rs: Simplified and combined existing sensor data structures, such as merging EkfNav1 and EkfNav2 into EkfNav, and Imu1 and Imu2 into Imu. [1] [2] [3]

Removing redundant code:

  • src/command.rs: Removed the CommandData struct and its new method, simplifying the Command enum. [1] [2]
  • src/sensor.rs: Removed the Sensor struct and its new method, along with other unused sensor data structures.

Dependency updates:

  • Cargo.toml: Added the ublox crate with serde feature for handling GPS data.

These changes streamline the codebase, improve message handling, and enhance the flexibility for future expansions.

@NoahSprenger NoahSprenger merged commit 4a6662e into master Nov 17, 2024
2 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.

1 participant