-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Create Smart Battery Module #3944
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
base: master
Are you sure you want to change the base?
Conversation
Right now there's a bunch of code for logging, might remove that later on or make it a toggle in settings |
I made logging an option in settings, app is good to go. This is my first module I've made, so if anything looks strange, I would love to get feedback on it :) |
I decided to try it out on physical hardware. Currently the behaviour is weird where it shows 0hr (logging off) but turning on loging gives me a figure ranging from 8hr to 99days (which i expected). Hopefully it is just a temp thing and in a day or so it starts working properly. We shall see P.S. Isnt logging just for debuging? Or does it actually have an effect on the module? |
@RelapsingCertainly Logging shouldn't change anything fundamental, you're right, it's just for debugging. Give it up to 2 days to see if the drainage rate is just fluctuating, and it's still learning from that. I would love to hear a status update on this later on :) PS: how did you install the app? If you copied the files into the ide and uploaded it, maybe something went awry there... |
@RelapsingCertainly I have it installed through my app loader on my watch, and it seems pretty steady and accurate, logging doesn't have an impact... If the problem is still there, try installing it with my app loader, and see if that fixes anything |
I did just download the files, rename them according to the metadata and uploaded them. I'll try to use the app loader in case that makes a difference |
Looks like it was an ide error. Its now saying 99days so it looks like estimates are at least being made |
@RelapsingCertainly Glad to hear! Let me know how it goes after a few days :) |
This provides a module an app can use for battery. It learns your usage patterns over time by averaging out drainage rates, and extrapolates that into a pretty accurate prediction.