-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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 MSP request for UBLOX SatInfo and disable SatInfo on ARMING. #13240
Conversation
Do you want to test this code? You can flash it directly from Betaflight Configurator:
WARNING: It may be unstable. Use only for testing! |
AUTOMERGE: (FAIL)
|
a646531
to
fb74f3c
Compare
Tested - works GREAT! Much faster than before. The only thing I haven't confirmed is that the sat info is not being sent when not connected to configurator. That requires parallel monitoring the output of the GPS module in pyGps or uCenter, and powering it up with Configurator not connected. Two questions:
It is very important that SatInfo is not requested in flight, it results in a very big serial data load. |
For your second question i think (for the little i understand code) is also checking the armed flag. So, in anycase, if you take off satinfo is disabled. |
3c68853
to
9e0b5b7
Compare
SatInfo will only be requested (enabled) when sending the MSP message on the gps tab in configurator.
ARMING the quad disables SatInfo. |
39d69a5
to
e600b11
Compare
e600b11
to
eefd3a8
Compare
eefd3a8
to
8d380c4
Compare
As configurator some times fails to get satellite info as it missed the opportunity we rather like to add a MSP request from client.