sungrow-hybrid: fix reset to normal battery mode

[hameno]

The previously used forced value does not work reliably on my SH5.0RT. I'm proposing using the initial implementation that I successfully tested locally.

[andig]

The go plugin has an inherent memory leak and should be avoided.

[hameno]

The go plugin has an inherent memory leak and should be avoided.

Any other suggestion on how to get the rated value?

[andig]

Not really at this time. Either accept the leak (battery won't be switched that often) or live with manual configuration.

The previously used forced value does not work reliably on my SH5.0RT

Unclear why that should be since it would write exactly the same value? I'm not convinced that the fix does fix what you think it does.

[hameno]

It would not write the same value, the BDC rated power is 5kW, this always tries to write 10kW. My battery is a SBR096 which is only rated for 5kW

[andig]

value: {{ .maxchargepower | int }} # W

It will write whatever you put into maxchargepower. Just set it to 5k?

[hameno]

value: {{ .maxchargepower | int }} # W

It will write whatever you put into maxchargepower. Just set it to 5k?

But that is only one of the cases here. And only after my change? Not sure I get your comment here. I specifically handle this existing advanced configuration option now and offer a safe default if not configured.

[andig]

It would not write the same value, the BDC rated power is 5kW, this always tries to write 10kW.

I don't understand. Why would it do that if you configure 5k? Why is the auto detection/ Go required?

[hameno]

I'm very confused.
Before this change: always write 10kW
After this change: allow overriding this or have it based on the actual limits of the battery. I'm simply using the same code that is used in other points of this template.

[andig]

Lets just do the override then, leave the auto option out.

[andig]

Bei maxchargepower sollte noch ein

usages: ["battery"]

eingefügt werden- in anderen Modi tut der Parameter ja nichts.