-
-
Notifications
You must be signed in to change notification settings - Fork 44
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
Potential optimisations around planned export to minimise writes #1681
Comments
Interesting I made a fix on 'main' for just these cases this morning, can you give it a try? |
Discharge fix on 'main' for testing |
Hi @springfall2008 , I've uploaded the logs for last night as it's probably easier for you to see the writes. It seems better, but 2 observations
|
I'm not sure that will do anything, 4: 1.05 means that you will discharge 5% faster than normal when your battery reaches 4%? Can you change metric_battery_scaling_discharge to adjust the predicted discharge speed up/down |
Firstly thank you for all the work you have done already in this area following the STTG video. I've noticed that the major contributor to writes is now the forced export at the end of my plan .prior to charging up on the cheap rate. It feels like this is an opportunity for further improvement and I wanted to share my thoughts based on observations.
Last night it required 23 write requests to discharge what remained in my battery, which seemed excessive. Ironically this did include my automation halting a spurious calibration at 23:10 (said to be triggered by too many writes 🥴)
I noticed the following contributors:
My thoughts for improvements
Not saying any of this is necessarily possible, but wanted to spark discussion
Logs from the period
write_log_inv_1.log
predbat1.log
The text was updated successfully, but these errors were encountered: