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

Initiate magnetometer based compass calibration from button presses #5553

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

Conversation

danwelch3
Copy link

  • only active for BMX160 accelerometers on RAK_4631
  • replace automatic calibration on power on with button triggered calibration
  • set 5 presses to trigger 30s calibration
  • set 6 presses to trigger 60s calibration (useful if unit is not handheld, ie vehicle mounted)
  • show calibration time remaining on calibration alert screen

@fifieldt
Copy link
Contributor

Thanks for this! It looks like you may have checked in changes to the protobufs. These are automatically generated , so please update the patch not to include these :)

- only active for BMX160 accelerometers on RAK_4631
- replace automatic calibration on power on with button triggered
  calibration
- set 5 presses to trigger 30s calibration
- set 6 presses to trigger 60s calibration (useful if unit is not
  handheld, ie vehicle mounted)
- show calibration time remaining on calibration alert screen
- exclude changes from non RAK 4631 builds
- remove calls to screen when not present
- exclude all compass heading updates on variant without screen
@danwelch3 danwelch3 force-pushed the feature-button-calibrate branch from 2f7e479 to 686f569 Compare December 12, 2024 22:11
@danwelch3
Copy link
Author

Thanks for pointing that out. I think I successfully removed the changes to protobufs from the patch.

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.

2 participants