forked from icoderaven/Firmware
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Upstream merge with PX4 #4
Open
kshitijgoel007
wants to merge
10,000
commits into
rislab:feature/hitl_upstream_merge
Choose a base branch
from
PX4:master
base: feature/hitl_upstream_merge
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Upstream merge with PX4 #4
kshitijgoel007
wants to merge
10,000
commits into
rislab:feature/hitl_upstream_merge
from
PX4:master
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
Is anything happening with this pull request? It might be a better idea to pull master into a branch here so that we have something static that we can control. Also, for anyone else getting way too many notifications from this: you can unsubscribe from the pull request. |
Adds COM_FLT_TIME_MAX param and logic in Commander to enforce RTL when flight time is above this value. User can only override to LAND mode, but not proceed flight beyond that. Signed-off-by: Silvan Fuhrer <[email protected]>
- add logic for detecting high wind speed in Commander, and handle it toghether with wind speed warning - trigger and enforce RTL if COM_WIND_MAX is breached Signed-off-by: Silvan Fuhrer <[email protected]>
Check during init to avoid out-of-bound access.
Same logic as the existing mixer. Untested.
The flight termination action on geofence violation is described as only trigger, when the corresponding circuit breaker is not disabled. However, the description of the circuit breaker states, that the geofence action is not depedning on this circuit breaker. The implementation follows the description of the circuit breaker. Hence the GF_ACTION description is adapted.
Did some prep work for redundant interfaces by introducing CanardHandle class to decouple physical interfaces from cyphal.cpp
…e sensors to save flash
- mavlink in PX4/Firmware (d6dff7d): mavlink/mavlink@3b52eac - mavlink current upstream: mavlink/mavlink@a1cb2c0 - Changes: mavlink/mavlink@3b52eac...a1cb2c0 a1cb2c0e 2022-05-05 KonradRudin - Figure eight MAV_CMD (#1831) 0ebdf846 2022-04-28 Hamish Willee - common.xml: Deprecate MAV_CMD_GET_MESSAGE_INTERVAL for MAV_CMD_REQUEST_MESSAGE. (#1835) f357fc78 2022-04-27 Hamish Willee - Deprecate MAV_CMD_GET_HOME_POSITION for MAV_CMD_REQUEST_MESSAGE (#1834)
- people are updating this without really understanding the purpose, and it's generally no longer useful
- allow saving initial gyro cal if very close to 0
…l pressure - disable CONFIG_SENSORS_VEHICLE_AIRSPEED on boards only used for multicopter
- run `sensors` hub to process sensor_mag and publish vehicle_magnetometer - update uavcannode to use vehicle_magnetometer
…mon sensors to save flash
Signed-off-by: Claudio Micheli <[email protected]>
Co-authored-by: Megan McCormick <[email protected]> Co-authored-by: Chuck Faber <[email protected]>
- all sources of optical flow publish sensor_optical_flow - sensor_optical_flow is aggregated by the sensors module, aligned with integrated gyro, and published as vehicle_optical_flow Co-authored-by: alexklimaj <[email protected]>
- remove internal accumulation and publish every valid raw sample synchronized with sensor - store timestamp_sample from motion interrupt - improve timing requirements from datasheet (minimum delays after register read/write)
- remove internal accumulation and publish every valid raw sample synchronized with sensor - store timestamp_sample from motion interrupt - improve timing requirements from datasheet (minimum delays after register read/write)
Co-authored-by: Matthias Grob <[email protected]>
Remove the entire external yaw handler, dynamic memory allocation, pointer passing logic. Directly instanciate the weather vane instance in the flight tasks that support it.
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.
On the HITL firmware branch. Doing this for Notch filter and other latest features.