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

Fix compass calibration interval timeout #3865

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

haslinghuis
Copy link
Member

@haslinghuis haslinghuis added this to the 10.10.0 milestone Mar 30, 2024
@haslinghuis haslinghuis self-assigned this Mar 30, 2024
Copy link

netlify bot commented Mar 30, 2024

Deploy Preview for origin-betaflight-configurator ready!

Name Link
🔨 Latest commit 70a4d2f
🔍 Latest deploy log https://app.netlify.com/sites/origin-betaflight-configurator/deploys/66083c8b5569070008a9c310
😎 Deploy Preview https://deploy-preview-3865--origin-betaflight-configurator.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

This comment has been minimized.

@haslinghuis haslinghuis force-pushed the fix-mag-calibration branch from ea4e3eb to 8c303fc Compare March 30, 2024 15:40

This comment has been minimized.

@haslinghuis haslinghuis force-pushed the fix-mag-calibration branch from 8c303fc to 44e0deb Compare March 30, 2024 16:10

This comment has been minimized.

@haslinghuis haslinghuis force-pushed the fix-mag-calibration branch from 44e0deb to 70a4d2f Compare March 30, 2024 16:23
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link
Contributor

Do you want to test this code? Here you have an automated build:
Betaflight-Configurator-Android
Betaflight-Configurator-Linux
Betaflight-Configurator-macOS
Betaflight-Configurator-Windows
WARNING: It may be unstable and result in corrupted configurations or data loss. Use only for testing!

Copy link
Member

@nerdCopter nerdCopter left a comment

Choose a reason for hiding this comment

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

approving untested

@ctzsnooze
Copy link
Member

ctzsnooze commented Apr 3, 2024

Works perfectly! I had issues with my local build with GitHub... now I tested this branch, it's great!

It would be even better if, on initiation (clicking the button), the text says, "Calibrate Magnetometer: waiting", or something like that. If no movement (ie if data collection never starts because movement threshold is not reached), then it times out at 15s and says "Calibrate Magnetometer: failed" in the button name, or something like that.

One the other hand, if it works out, then once the cal is complete, we can say "Calibrate Magnetometer: success!"

The button text could reset every time we enter the screen.

@haslinghuis
Copy link
Member Author

haslinghuis commented Apr 3, 2024

Only have the CALIB armingDisableFlag as a measure to control the button. tCal in firmware would reset that flag.

In my testing the button would resolve in 15 seconds when no movements were registered.
When gyro movement was registered it allowed the calibration process to complete.

@ctzsnooze
Copy link
Member

@haslinghuis sorry my bad, initial test was accidentally with a build from Master. Now I tested this branch correctly, it works great. Approved.

@haslinghuis haslinghuis merged commit 146fb04 into betaflight:master Apr 4, 2024
12 checks passed
@haslinghuis haslinghuis deleted the fix-mag-calibration branch April 4, 2024 09:46
chmelevskij pushed a commit to chmelevskij/betaflight-configurator that referenced this pull request Apr 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: COMPLETED
Development

Successfully merging this pull request may close these issues.

3 participants