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

current meter scale to 500 #9703

Merged

Conversation

P-I-Engineer
Copy link
Contributor

current meter scale was incorrect. i have this aio for testing.

@lida2003
Copy link
Contributor

lida2003 commented Feb 7, 2024

current meter scale was incorrect. i have this aio for testing.

@P-I-Engineer Thanks for you PR. Currently, we are on Chinese Year Holiday. We can't look the data (in lab) before Feb 17.

Can you share your test data when configure scale 500?

I think it might be poorly calibrated. So we will check and re-do the calibration on Feb 17 back from holiday.

Ref: BetaFlight Current Calibration Guide

@P-I-Engineer
Copy link
Contributor Author

inav current meter scale: 250
aocoda recommended scale: 540
betaflight config scale: 500

i flew with the scale at 500. i consumed 156mah per my telemetry, i charged 156mah back in. Obviously the number being the same is coincidental. But matches up way better than double the consumption when using a scale of 250.

@lida2003
Copy link
Contributor

lida2003 commented Feb 7, 2024

aocoda recommended scale: 540

Well, I'm NOT sure if there is any suggestion comes from us. But I indeed checked bf configurations, which is 500.

As I didn't see any report or data for this AIO board right now. We'll check again, and then come back to the issue.

@DzikuVx DzikuVx added this to the 8.0 milestone Feb 8, 2024
@DzikuVx DzikuVx merged commit f2f5356 into iNavFlight:master Feb 8, 2024
14 checks passed
@lida2003
Copy link
Contributor

@P-I-Engineer

We have did current calibration, and the result as below.

PR will be provided later.

BetaFlight/iNav Current Calibration Guide

图片

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.

3 participants