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

[GPS] Add SEA and MOWER dynamic models. #10211

Merged
merged 2 commits into from
Jul 9, 2024
Merged

[GPS] Add SEA and MOWER dynamic models. #10211

merged 2 commits into from
Jul 9, 2024

Conversation

mmosca
Copy link
Collaborator

@mmosca mmosca commented Jul 8, 2024

Potentially better models for hovers.

Not a lot of documentation about the different dynamic models.

Potentially better models for hovers
@sensei-hacker
Copy link
Collaborator

Mower?

download (19)

@mmosca
Copy link
Collaborator Author

mmosca commented Jul 8, 2024

You get the idea, but this is a thing: https://www.ardumower.de/en/home.html

But in great ublox tradition... m10 docs says mower is not always available.

m8 and m9 made no mention of that.

@Jetrell
Copy link

Jetrell commented Jul 8, 2024

@mmosca The idea seems good https://www.u-blox.com/en/high-precision-openmower

As far as optimally using the benefits of the dynamic model types, under all situations.
It maybe an idea to add gps_dyn_model to the programming tab. So users can base a dynamic model selection, on a GPS speed setpoint.
e.g. use air_2G for speeds greater than 15km/h. And switch to mower or sea when the copter is slowing to a hover.

@mmosca
Copy link
Collaborator Author

mmosca commented Jul 9, 2024

Changing the dyn model on the fly may be trickier than it looks.

Currently we only setup the gps on start up.

The ublox command itself is simple to implement, and changing modes does not require a gps reboot. I haven't played with adding new programming framework features, but if it can trigger the action when the state changes, it should simplify things.

@mmosca mmosca merged commit 76d18b2 into master Jul 9, 2024
23 checks passed
@DzikuVx DzikuVx added this to the 8.0 milestone Jul 12, 2024
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.

4 participants