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 configuration for dynamic platform and measurement rate #55

Merged
merged 7 commits into from
Jul 1, 2024

Conversation

AngusJull
Copy link
Contributor

Added more configuration options, increasing the GPS update rate to 3Hz (as opposed to 1Hz). Disabled the BSD constellation (as GPS + GALILEO should be enough, and allows a higher update rate). Considering the datasheet, we should be able to poll at 10Hz, which is possible for a second or two before the sensor stops responding to poll requests until reconfigured for a lower measurement rate.

Ensuring more constellations are disabled or using periodic messages may allow the increase of the measurement rate.

Closes #44

Added configuration for the platform model, which will prevent gps data
from being discarded during flight, and increased the mesasurement rate
to roughly 3Hz
It seems that using the hypothetical limit on update rate for NAV-PVT
causes problems with messages after about 10 have been recieved. This
might be a problem with periodic/polled messages. Also fixed the config
key, and moved some definitions

/** A configuration key for enabling or disabling output of NMEA messages on I2C */
#define NMEA_I2C_OUTPUT_CONFIG_KEY 0x10720002
/** The confirmation value for the platform model that corresponds to an airborne vehicle doing <4G of acceleration */
Copy link
Collaborator

Choose a reason for hiding this comment

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

RIP that this is the best g-force we can get. Leaving this comment just to document that.

Copy link
Collaborator

@linguini1 linguini1 left a comment

Choose a reason for hiding this comment

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

LGTM! Waiting to merge for testing.

@linguini1 linguini1 merged commit 5af1fc1 into main Jul 1, 2024
1 check passed
@linguini1 linguini1 deleted the m10spg-config branch July 1, 2024 22:54
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.

Add configuration options for the M10SPG
2 participants