2.0.1 - Energy from Yesterday reported

hultenvp / solis-sensor

HomeAssistant integration for the SolisCloud PV Monitoring portal via SolisCloud API

Apache License 2.0

191 stars 42 forks source link

2.0.1 - Energy from Yesterday reported #76

Closed henrikeri closed 2 years ago

henrikeri commented 2 years ago

Describe the bug 2.0.1 Still reports energy from yesterday. However, today only energy from 1 repeater was reported if this is of any help.

Config sensor:

platform: solis name: "Inverter" portal_domain: "m.ginlong.com" portal_username: !secret solisUser portal_password: !secret solisPass portal_plant_id: !secret solisPlant

Versions HA Version: core-2022.2.0b3, Home Assistant OS 7.2 HACS 1.20.0 Solis 2.0.1

Debug logs No log issue reflects what is happening.

Screenshots bug

hultenvp commented 2 years ago

Hmmm, this is....interesting. The fix works on my inverter, but apparently still some corner case I overlooked. More leads are welcome so that I can try to reproduce.

Siwek666 commented 2 years ago

Works perfect for me i think with solarman platform integration.. Thanks so much.

Versions HA Version: core-2021.12.10, Home Assistant OS 7.2 HACS 1.20.0 Solis 2.0.1

hultenvp commented 2 years ago

Ugghhh, I think I found it. Line 153 in service.py contains a wrong test; '<' should have been '>'. As a result the test evaluates wrong and returns for one measurement still the energy of yesterday. No clue why the problem went away with me and you still see it. Running tests now.

henrikeri commented 2 years ago

I have changed the line as well. Will report results tomorrow around noon when it is up and running. It also only seems to get 1 of the DC lines PV1, not PV2 of my multi string inverter. Is this related to the same?

hultenvp commented 2 years ago

Thanks looking forward to the result. I tested it, looks to be okay now, but I want to see if it doesn't break anything unexpected tomorrow when the sun comes up

Your other issue seems unrelated. Could you get a debug log and create a new issue for that?

Cheers

henrikeri commented 2 years ago

@hultenvp, the change seems to work at my end as well. It showed up as intended after sunrise this morning. As for the issue with PV2, it is once more showing.....

hultenvp commented 2 years ago

Thanks for letting me know this fix works, please open an issue for the PV2, but please add debug logging as I'm only having one string here.

hultenvp commented 2 years ago

Released in v2.0.2. Let's pray it is now really fixed

cpuks commented 2 years ago

Just had this issue again on 2.0.6 - whole yesterday production added to todays morning.

solar

kazikskrz commented 2 years ago

I'm having the same here. Every few days I get production value from day before. version 2.1.0 HA 2022.3.5 2022-03-28_08h35_43 2022-03-28_08h34_44 2022-03-28_08h33_35