evcc-io
Sungrow-hybrid: fix battery reset
Hello!
I think there is still an issue in setting 33047 to maxchargepower in 'normal' mode. This should just be the max value of the reg I believe.
H
Why doesn‘t stopping the mode in https://github.com/evcc-io/evcc/pull/24022/files#diff-0b0f0388fa0ea5962b68de7565e1131be1912c96e6d198ad57f6b53d77775b17R228 suffice?
Why doesn‘t stopping the mode in https://github.com/evcc-io/evcc/pull/24022/files#diff-0b0f0388fa0ea5962b68de7565e1131be1912c96e6d198ad57f6b53d77775b17R228 suffice?
It should, and it does work on my system. But others seem to have issues. However discharging battery to net is a different setting for the same register, one that is not used/set by EVCC so not sure the problem is created by evcc.
Changing 33047 is definetly not needed coming from forced battery charging mode. It is needed only when coming from hold mode, but then the value set should not be maxchargepower as this limits 'normal' discharging of battery.
Why doesn‘t stopping the mode in https://github.com/evcc-io/evcc/pull/24022/files#diff-0b0f0388fa0ea5962b68de7565e1131be1912c96e6d198ad57f6b53d77775b17R228 suffice?
It should, and it does work on my system. But others seem to have issues. However discharging battery to net is a different setting for the same register, one that is not used/set by EVCC so not sure the problem is created by evcc.
Changing 33047 is definetly not needed coming from forced battery charging mode. It is needed only when coming from hold mode, but then the value set should not be maxchargepower as this limits 'normal' discharging of battery.
+1
can i disable this sequence? I have big problems because of this behaviour, after ending "charging batteri from grid" reseting "max discharge" 33047 value is really not something i want...
Ideally it would need to be reverted to whatever it was before the mode change. But not sure if that is even feasible ...
Hilft es, ein warten auf... "feedback" und "test" label zu haben? Ist Test nicht eine Art von Feedback? Es wäre super, wenn wir keine neuen Labels einführen müssten...
@premultiply how can we advance on this? We would like to test the inversion of the first commands, so change ems mode last when returning to normal battery mode. Do you modify your PR or do I create a new one?
duplicate of https://github.com/evcc-io/evcc/pull/25011 will continue there
The change of order should be sufficient to fix https://github.com/evcc-io/evcc/issues/23918 No need to reset reg 13051. I had another fix (reset of max discharge power) in my PR. Should I include it here?