-
-
Notifications
You must be signed in to change notification settings - Fork 81
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 formula #486
base: master
Are you sure you want to change the base?
Fix formula #486
Conversation
✅ Deploy Preview for origin-betaflight-com ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
approval for workflow, but formula still in discussion in discord. |
Do not merge yet. According to code it should be something like Which I think should give int32_t millivolts = ((uint32_t)src * getVrefMv()) / 4096;
// y=x/m+b m is scale in (mV/10A) and b is offset in (mA)
int32_t centiAmps = (millivolts * 10000 / (int32_t)config->scale + (int32_t)config->offset) / 10; |
fe3c828
to
1fb999f
Compare
1fb999f
to
54f23a0
Compare
Quality Gate passedIssues Measures |
According to code it should be
But this one from @KarateBrot looks cleaner