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

Update the Heltec board battery level read accuracy. #3955

Merged
merged 5 commits into from
May 31, 2024
Merged

Update the Heltec board battery level read accuracy. #3955

merged 5 commits into from
May 31, 2024

Conversation

Heltec-Aaron-Lee
Copy link
Contributor

Update the Heltec board battery voltage read parameter.

Update the Heltec board battery voltage read parameter.
Update the Heltec board battery voltage read parameter.
Update the Heltec board battery voltage read parameter.
Update the Heltec board battery voltage read parameter.
@CLAassistant
Copy link

CLAassistant commented May 22, 2024

CLA assistant check
All committers have signed the CLA.

@HarukiToreda
Copy link
Contributor

HarukiToreda commented May 23, 2024

Update the Heltec board battery voltage read parameter.

Hey Aaron, you need to sign the CLA

@Cisien
Copy link

Cisien commented May 23, 2024

I can confirm these values work as an override in the current firmware by setting the override to 5.1205 (4.9*1.045)

@mkgin
Copy link
Contributor

mkgin commented May 23, 2024

"*1.045" 5 bytes of bureaucracy...

@garthvh
Copy link
Member

garthvh commented May 30, 2024

Hi @Heltec-Aaron-Lee we really need access to the fork of meshtastic you are giving out to customers, needs to be available per the GPL3, it appears that the protobufs are out of sync and this is becoming a support issue. We also need the CLA signed before this can get merged.

@Heltec-Aaron-Lee
Copy link
Contributor Author

Hi @garthvh sorry for the delay. It is already signed, I will pay attention to it in the future.

@thebentern thebentern merged commit b9edc75 into meshtastic:master May 31, 2024
80 of 81 checks passed
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.

7 participants