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
Premature triggering of EOC #35
Comments
@luckylinux, I have been testing a cycle / time threshold for triggering EOC, you can see the change to the code at d4f257c Please note that this is currently a 60 cycle threshold and this section of code is generally run once per second. |
Please note that this logic is liable to change in the future packaged version under development. |
But it's not merged in main branch yet (I assume this is in the development branch). Please make it a configurable parameter. I think also chaosnature asked for a "wait at least X seconds before leaving bulk / top balance". I would put it to 15-30 minutes ... |
You can see it in https://github.com/Sleeper85/esphome-jk-bms-can/compare/35-premature-triggering-of-eoc, applied to a single YAML for now. If you can test the code first with the default 60 cycles, that would be appreciated. I would caution against a period as long as 30 minutes however, that will likely overcharge your cells as per the detail in https://nordkyndesign.com/charging-marine-lithium-battery-banks/. |
OK so basically backport the "delta" into my version, got it. What do you mean by "overcharge your cells" ? Isn't that how top balance works anyways ? And as long as all cells < 3.65 VDC it shouldn't overcharge ... |
@MrPabloUK OK I implemented your changes in my file (including dynamic change possibilities via Home Assistant) now and reflashed: https://github.com/luckylinux/jk-bms-build-helpers/blob/main/esphome-jk-bms-can/esp32-ble-1.17.4.yaml |
@MrPabloUK: What would happen if now, I disable BLE connection from ESP32 to JK BMS (concerning CAN Communication) ? Would it cause the inverter to trip (BMS_Err_Stop is SET now). I'd consider changing the Balance Voltage to 3.43V as you suggested now, but I need to use the JK BMS APP for that. |
Honestly, I don't know for certain. I use a wired connection as I don't quite trust BLE for this project. |
Question is ... If I were to implement multi BMS like Der_Hannes did, should I use this as a base or the development branch ? |
Closing issue as feature has been added to V1.17.5 |
Please note that Balancing is ON yet the ESP32 went into FLOAT charging status now.
Probably this happens because for a second or so (probably battery discharged for a few seconds) the cell voltage went below balance trigger voltage, due to the battery being discharged (either because of clouds or load step):
The text was updated successfully, but these errors were encountered: