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

accel and gyro calibration cleanup #14237

Closed
wants to merge 2 commits into from

Conversation

dagar
Copy link
Member

@dagar dagar commented Feb 25, 2020

  • kill IOCTLs for accel and gyro calibration
  • move CAL parameter loading to sensors module

Other than cleanup, the motivation for this change is to allow the filtered angular velocity (and acceleration) to be computed directly from the raw sensor data array (FIFO) when it's available.

@dagar dagar force-pushed the pr-sensors_accel_gyro_calibration_cleanup branch from 4824993 to 9b12e30 Compare February 25, 2020 18:29
@@ -52,6 +52,14 @@ class ModuleParams : public ListNode<ModuleParams *>
setParent(parent);
}

// empty copy and move constructors (don't copy _children)
ModuleParams(const ModuleParams &) {};
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TODO: review

@dagar dagar force-pushed the pr-sensors_accel_gyro_calibration_cleanup branch from f8d0d77 to 30132a1 Compare March 2, 2020 23:41
@dagar dagar force-pushed the pr-sensors_accel_gyro_calibration_cleanup branch 8 times, most recently from 1d1a985 to 4ec552a Compare March 12, 2020 13:50
@dagar dagar added this to the Release v1.12.0 milestone Mar 21, 2020
@dagar dagar self-assigned this Mar 21, 2020
@dagar dagar force-pushed the pr-sensors_accel_gyro_calibration_cleanup branch 2 times, most recently from 1394c1f to a7e1b82 Compare April 25, 2020 13:51
dagar added 2 commits April 26, 2020 13:30
 - kill IOCTLs for accel and gyro calibration
 - move accel/calibration entirely to sensors module
@dagar dagar force-pushed the pr-sensors_accel_gyro_calibration_cleanup branch from a7e1b82 to 2f63346 Compare April 27, 2020 00:45
@dagar dagar closed this Apr 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant