-
Notifications
You must be signed in to change notification settings - Fork 13.4k
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
move IMU integration to sensors/vehicle_imu to fix potential accel/gyro sync issues #14906
Merged
Merged
Changes from 7 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
2906f40
sensors/vehicle_imu: integrate accel and gyro
dagar 9ba72f7
lockstep testing hacks
dagar 792b157
Merge remote-tracking branch 'px4/master' into pr-sensors_vehicle_imu…
dagar 3bef9c3
changes from code review
dagar eeecdc5
Integrator move from lib/drivers/device -> sensors/vehicle_imu
dagar 859b1eb
advertise vehicle_imu_status immediately
dagar b3f8ed7
Merge remote-tracking branch 'px4/master' into pr-sensors_vehicle_imu…
dagar 3cc350d
Merge remote-tracking branch 'px4/master' into pr-sensors_vehicle_imu…
dagar f645b3b
sensors: create VehicleIMU instances sequentially as data is actually…
dagar ff27251
new icm20948 remove obsolete PX4Accelerometer/PX4Gyroscope set_update…
dagar a1be128
Merge remote-tracking branch 'px4/master' into pr-sensors_vehicle_imu…
dagar c4b6c72
sensors/vehicle_imu: integrate gyro first, then only take accel until…
dagar d20e341
sensors/vehicle_imu: remove debug perf counters
dagar 09b3dab
Update src/modules/sensors/vehicle_imu/VehicleIMU.cpp
dagar 680de1a
Merge remote-tracking branch 'px4/master' into pr-sensors_vehicle_imu…
dagar e0180f3
sensors/vehicle_imu: don't integrate every available gyro sample unle…
dagar 470f418
TESTING
dagar 5f07340
Merge remote-tracking branch 'px4/master' into pr-sensors_vehicle_imu…
dagar 8c0a3d6
logger: limit vehicle_imu
dagar ba2fd75
vehicle_imu error counts
dagar 9c0f8c5
fix typo and formatting
dagar ebee09b
fix sensor_selection publication and add backup schedules
dagar ec5a35f
ekf2: remove accel/gyro bias reset warning (too noisy)
dagar 1b2769e
sensors status cleanup whitespace and ordering
dagar 44ec4ea
IMU_INTEG_RATE add enums and comment
dagar 043f83a
sensors status minor tweaks
dagar 90c1864
remove IMU_INTEG_RATE enums
dagar e463c7e
Merge remote-tracking branch 'px4/master' into pr-sensors_vehicle_imu…
dagar File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,16 @@ | ||
uint64 timestamp # time since system start (microseconds) | ||
|
||
uint32 accel_device_id # unique device ID for the sensor that does not change between power cycles | ||
uint32 gyro_device_id # unique device ID for the sensor that does not change between power cycles | ||
|
||
uint32[3] accel_clipping # clipping per axis | ||
|
||
uint32 accel_error_count | ||
uint32 gyro_error_count | ||
|
||
uint16 accel_rate_hz | ||
uint16 gyro_rate_hz | ||
|
||
float32 accel_vibration_metric # high frequency vibration level in the IMU delta velocity data (m/s) | ||
float32 gyro_vibration_metric # high frequency vibration level in the IMU delta velocity data (m/s) | ||
float32 gyro_coning_vibration # Level of coning vibration in the IMU delta angles (rad^2) |
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
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
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
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
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
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
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
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dagar can you rearrange the ID's so we don't have unused numbers? Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shift the entire set after 118 down? What's the expectation/requirement for these id numbers (persistence, etc)?