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

Merge px4 1.13.2 #4

Open
wants to merge 120 commits into
base: master
Choose a base branch
from
Open

Merge px4 1.13.2 #4

wants to merge 120 commits into from

Conversation

dev10110
Copy link
Member

@dev10110 dev10110 commented Feb 4, 2023

This branch works, and it has been updated with the px4 v1.13.2

I propose we replace current master with this branch

it has been tested in sim and in experiments on a quad.

probably worth checking if this still works on a rover before.

sfuhrer and others added 30 commits April 23, 2022 10:29
Weather vane should only set a yawrate setpoint, but no yaw setpoint.
Setting it to NAN when WV is active makes sure that whatever _yaw_setpoint
is set previously (e.g. through Waypoint) is not used.

Signed-off-by: Silvan Fuhrer <[email protected]>
 - on some H7 boards (cuav x7pro tested) there's an occasional hard fault when starting the mavlink shell that is no longer reproducible with the slightly expanded locking
 - this is likely just changing the timing (holding the sched lock for longer), but this should be harmless for now until we can identify the root cause
when the vertical pos or vel innov ratio is above the threshold, the
other one needs to be significant too and not just positive to trigger
the failure
… and cleanup

 - split ms4525_airspeed into separate ms4515 and ms4525 drivers
vincentpoont2 and others added 28 commits September 27, 2022 05:45
My assumption is that the bus are numbered < 127.
This saves about 100 bytes of flash.
As I understand it, only Rev 1 and Rev 2 were shipped by HB, and likely
to be used for the Mini and CM4.
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
…ator::Navigator()

This fixes the issue where the init happended in the initializer list, at which point
the params were not yet initialized and thus resulted in random values for the init
values of _mission_item.loiter_radius and .acceptance_radius.

Signed-off-by: Silvan Fuhrer <[email protected]>
…ight mag cal

 - preflight mag bias estimate is in the vehicle body frame and removed from the raw mag data after it's rotated, calibrated, etc
 - in flight mag bias (from ekf2) is in the vehicle body frame, but stored as a sensor frame offset immediately
to avoid float rounding errors leading to tiny acceptance radii
getting considered
…r takeoff items

otherwise a previously adjusted or uninitialized radius from the last flight
can cause problems during the new takeoff
@dev10110
Copy link
Member Author

dev10110 commented Feb 4, 2023

@hardikparwana whenever you are done with RSS, lets talk about making this our main branch.

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.