hoylabs / OpenDTU-OnBattery

Software for ESP32 to talk to Hoymiles/TSUN/Solenso Inverters, VE.Direct devices, battery management systems, and related peripherals
GNU General Public License v2.0
308 stars 66 forks source link

[Request] AC charger: Implement DC/Battery stop SoC #754

Closed gitisgreat2023 closed 6 months ago

gitisgreat2023 commented 7 months ago

Is your feature request related to a problem? Please describe.

Currently stopping of the charging is defined by voltage. As both the start and stop threshold conditions are defined in SoC %s, adding a target to stop charging would make sense.

Describe the solution you'd like

Adding Battery SoC - Stop charging.

To avoid confusion: best rename a) Battery SoC Stop threshold to Battery SoC - Stop discharging b) Battery SoC - Start charging to Battery SoC - Allow discharging

Describe alternatives you've considered

No response

Additional context

No response

Manos1966 commented 7 months ago

Quick question for my understanding: "Adding Battery SoC - Stop charging."

What are you using for charging? Because as I understand, the VICTRON MPPTs are not receiving commands from OpenDTU (in order to stop charging)

schlimmchen commented 7 months ago

There is no "stop charging". There is only "start discharging", or in other words "start using the battery for energy production".

I don't understand what you are asking for.

What should happen if the newly proposed "Battery SoC - Stop charging" threshold is reached?

schlimmchen commented 7 months ago

Is this about the Huawei charger?

gitisgreat2023 commented 7 months ago

Yes, after reaching the "Battery SoC - Stop charging" threshold it should stop charging...

Currently there are two SoC values: stop discharge, start again discharging. For example 15% (stop) and after reaching 20% allow discharging again.

The third value would be the upper level to stop charging, like 90% or 95%.

Yes my setup is a HM, Huawei, Pylontech setup.

schlimmchen commented 6 months ago

I think this is completed with #848, which will be included in the next release.

image

Please re-open this issue if I got it wrong.

github-actions[bot] commented 5 months ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.