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

Battery voltage sensor stops working if battery voltage drops below around 3 volts #14

Open
HanspeterH opened this issue Feb 21, 2018 · 7 comments
Assignees

Comments

@HanspeterH
Copy link

After the battery voltage has dropped to some voltage (e.g. 2.9V), the readout always returns the same value for all following messages sent (hundreds) although the DMM measured battery voltage is much below the reported fixed value of 3280mV. Probably rather a HW problem. But the SW should not send wrong data. This makes the sensor useless.
22:30:571763
dev id:ttn-node-01payload:0CD0001008E9battery:3280event:"motion"light:16temperature:22.81
... hundreds of messages later..
11:26:122514
dev id:ttn-node-01payload:0CD6002908E3battery:3280event:"button"light:41temperature:22.75
actual DMM measured battery voltage: 2.0V
In this case the SW should somehow indicate in the message, that the data is invalid.

@johanstokking
Copy link
Member

What is the code to read the battery level? Have you seen the new battery level LPP example?

@johanstokking johanstokking self-assigned this Feb 21, 2018
@HanspeterH
Copy link
Author

HanspeterH commented Feb 21, 2018

Yes I have seen it and charles seems to have included the new battery voltage readout logic already in the LPP sketch. Hopefully the sleep mode logic and the new battery voltage readout logic will be implemented in the basic sketch as well. I will test with the LPP sketch how long the ttn node will live with the improved code as soon as the ttn node with basic sketch is no longer sending its data, now running at 1.9V measured battery voltage and sensor data still looking good except voltage

@johanstokking
Copy link
Member

Great. Can you file a PR for the basic sketch if that's necessary?

@HanspeterH
Copy link
Author

Hmm I tried to read about PRs but I am lacking the skills about github terms and logic. I got stuck at the very beginning about selection of a base branch. I simply do not know all those terms as an end user and being only an occasional programmer. If I would continue i am pretty sure to do something wrong or useless...

@johanstokking
Copy link
Member

@HanspeterH there are good tutorials, for example https://www.digitalocean.com/community/tutorials/how-to-create-a-pull-request-on-github

Very useful skill!

@HanspeterH
Copy link
Author

Thanks, I will have a look...

@hallard
Copy link
Contributor

hallard commented Mar 1, 2018

Fixed with PR #15 ;-)

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

No branches or pull requests

3 participants