kellerza / sunsynk

Deye/Sunsynk Inverter Python library and Home Assistant OS Addon
https://kellerza.github.io/sunsynk/
MIT License
209 stars 88 forks source link

fix definition selection logic introduced when implementing hv support #269

Closed rixxxx closed 5 months ago

rixxxx commented 5 months ago

I made logic error when implementing 3PH HV support. Now 3PH LV gets both 3PH LV and 1PH definitions applied.

Currently:

sunsynk-1  | 08:58:17 INFO    Using three phase sensor definitions.
sunsynk-1  | 08:58:17 INFO    Using Single phase sensor definitions.

With the patch:

sunsynk-1  | 10:11:11 INFO    Using three phase sensor definitions.