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
The latest QGroundControl and the latest PX4 firmware parameters do not match. #11252
Comments
In the latest version of the PX4 firmware, there have been changes to some parameters. The parameter FW_ARSP_MODE has been renamed to FW_USE_AIRSPD, and the CBRK_AIRSPD_CHK parameter has been replaced with SYS_HAS_NUM_ASPD. For operations without an airspeed sensor, you would set SYS_HAS_NUM_ASPD to 0. Similarly, if you do not wish to use the airspeed sensor data in the controller, you would set FW_USE_AIRSPD to 0 |
This is going to require version check on parameter access for forward/back compatibility |
QGC shouldn't have to worry about |
Correct.
But what happens if the airspeed sensor hasn't been calibrated yet. Will that still work? |
Currently yes, though we're working on enabling multi airspeed sensors which would then require an adapted workflow in QGC again. @DonLakeFlyer what do you do for the Mag calibration in QGC to figure out when to display it as "mag needs calibration"? |
|
@dagouxiong233 @sfuhrer Can you both review the changes in QGC to makes sure I have this version specific handling correct? Specifically these two methods:
|
@DenysZaytsev Once CI finishes can you test this? You can pull the install artifacts from the GitHub Actions. |
Yes, I can test with Matek ASPD4525 |
@DenysZaytsev Builds are ready. What OS do you use? I can point you to the install. |
@DonLakeFlyer I've commented on the PR. Thanks a lot for addressing it! |
Current Behavior
1:FW_ARSP_MODE, 1:CBRK_AIRSPD_CHK
System Information
The text was updated successfully, but these errors were encountered: