wills106 / homeassistant-solax-modbus

SolaX Power Modbus custom_component for Home Assistant (Supports some Ginlong Solis, Growatt, Sofar Solar, TIGO TSI & Qcells Q.Volt Hyb)
310 stars 95 forks source link

pv_power_total unit of measurement Sofar HYD 10ktl-3ph #244

Closed juraurban closed 1 year ago

juraurban commented 1 year ago

PW Power total sensor has wrong decimal value.

image

Setting scale = 0.1 at this register, helps.

image

Thank you for fixing that in your code as well.

wills106 commented 1 year ago

Are you on 2022.12.x I thought I corrected that during 0.7.0bx before releasing the stable branch?

juraurban commented 1 year ago

I am on latest version. If you have a look into current code of plugin_sofar, the wrong scale must have gotten there back again during some latestupdates:

image
wills106 commented 1 year ago

With scale at 0.1 does it ever return 2 digits after the decimal ie 2.34kW or is it only ever 2.3kW? I might need to drop the rounding to 1 as well if it's ever only 2.3

wills106 commented 1 year ago

PV Power Total is corrected in 2022.12.6

wills106 commented 1 year ago

Have you managed to test this? If so and it's correct I'll close the issue down.

darkrain-nl commented 1 year ago

For my device the 0.01 and rouding 2 were working fine, now I get this: afbeelding

Could that be because I don't have any panels connected to PV2?

wills106 commented 1 year ago

I have just checked and you have a PV only Inverter 'SS2E' don't you?

So it must have a slightly different rounding on PV Power Total

darkrain-nl commented 1 year ago

I have just checked and you have a PV only Inverter 'SS2E' don't you?

So it must have a slightly different rounding on PV Power Total

Yes that is correct, is there any smart way to fix that? I do have a workaround and that is to use a different sensor for this reading, I just use sensor.sofar_active_power_output_total which seems to be the same thing anyway in my specific case.

juraurban commented 1 year ago

Sorry for late answer. Unfortunately I have had panels covered with layer of snow since last week therefore 0. Shows only 1 decimal ![Uploading 7AFDE8E1-F170-4C94-BC83-CF06A90A649E.png…]()

wills106 commented 1 year ago

Yes that is correct, is there any smart way to fix that?

Yes I can have a PV only version and a Hybrid only version of the same sensor with the different scaling required.

juraurban commented 1 year ago

ACE83DAC-590C-4B64-BBF1-5E29FA1748C3

juraurban commented 1 year ago

428E2BDD-8D0E-4678-A8FF-F1DAA317B068

juraurban commented 1 year ago

Inverter shows 10W current production

wills106 commented 1 year ago

As PV Power Total is only designed from Sofar to be in 0.1 resolution it won't display anything less than 100w

wills106 commented 1 year ago

2022.12.8 will correct "PV Power Total" scaling on KTLX-G3 Inverters

darkrain-nl commented 1 year ago

2022.12.8 will correct "PV Power Total" scaling on KTLX-G3 Inverters

Thanks, can confirm this works great!

wills106 commented 1 year ago

I'm going to close this down now as "PV Power Total" is now correct for the Hybrid and PV only Inverters.