forked from PX4/PX4-Autopilot
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Bring Open-UAS/PX4-Autopilot:stable up-to-date with the upstream PX4/PX4-Autopilot:stable #12
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- temporary solution to flash overflow
- sensor_baro.msg use SI (pressure in Pascals) - update all barometer drivers to publish directly and remove PX4Barometer helper - introduce baro cal (offset) mainly as a mechanism to adjust relative priority - commander: add simple baro cal that sets baro offsets to align with GPS altitude (if available) - create new sensors_status.msg to generalize sensor reporting
…nor updates to stay in sync with air data
- for ublox this corresponds to NAV_RELPOSNED
Signed-off-by: RomanBapst <[email protected]>
…nvalid Command denied is defined as "supported but has invalid parameters" which matches the case.
Co-authored-by: Alex Mikhalev <[email protected]>
This fixes unspecified behaviour due to evaluation order of args. It's up to the compiler whether next_number() or next_dots() is executed first which means that the behaviour is not properly specified.
As I understand it, only Rev 3 and Rev 4 were shipped by HB for Mini and CM4, and are likely to be used for it. It would be nice to have all combinations but it requires quite some flash in the current implementation.
- Removed commented out config data. - hw_mft_list_v0540 was the same as hw_mft_list_v0500
- alias from hw_mft_list_v0650 to hw_mft_list_v0600 as it is the same - add V6X50 again
This fixes the errors showing up at startup for me with a Black Cube.
This was forgotton with all the merges and shuffling previously.
This fixes a Python 3.10 issue for me.
Before this fix, this function would stall and somehow never return.
Clear the calibration at the end only otherwise everything gets erased at the end of the first iteration of the outer loop
This prevents publishing a negative pressure which leads to a NAN altitude estimate further down the line.
- vehicle_imu_status can publish immediately on any measured sample rate change or sensor error increment, but the windowed mean/variance shouldn't necessarily reset that often
This was referenced Oct 12, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Before bringing Open-UAS/PX4-Autopilot:stable up-to-date, we must first update Open-UAS/PX4-SITL_gazebo:master to the current version used by PX4/PX4-Autopilot:stable. There is already an open pull request to resolve this issue, please see Open-UAS/PX4-SITL_gazebo#2.
#11 will be closed without merging as this pull request serves the same purpose, the only difference being that here merge conflicts have been resolved.
Note: Please select 'Squash and merge' when merging.