JensenNick / huawei_solar_pees

Power, Energy and Economy Sensors in Home Assistant
MIT License
33 stars 3 forks source link

Error corrected - State Class for PV Economy Sensors #17

Closed JensenNick closed 5 months ago

JensenNick commented 5 months ago

Discussed in https://github.com/JensenNick/huawei_solar_pees/discussions/15

Originally posted by **JensenNick** January 28, 2024 Error corrected: State class total_increasing corrected to total for all pv_economy sensors. Total_increasing is not supported for state_class: monetary. History log: cc830b31fa3a558599f139141ed3010df1119e0e ![2024-01-28_07-35-17](https://github.com/JensenNick/huawei_solar_pees/assets/149193835/1eeb803e-6c72-4d47-b71f-e9ee687ca9ca)
JensenNick commented 5 months ago

I'm onto something here. It seams like when you do an one time install HA is handeling out names in an way which I did not intend / anticipate. It seams like the error is provoked by that some sensor use the same prefix "_2" as HA does.

JacobsenKim commented 5 months ago

min ser sådan her ud

Skærmbillede fra 2024-01-29 13-34-48

Virker til den er gået død for mig den 26 omkring kl 6:00

JensenNick commented 5 months ago

Virker til den er gået død for mig den 26 omkring kl 6:00

@JacobsenKim Tror ikke at det er samme issue, men jeg har også svært ved at se at PEES skulle være årsagen. Er du evt- løbet tør for diskspace i din database?

JacobsenKim commented 5 months ago

Brugt lagringsplads 75.3 %

JacobsenKim commented 5 months ago

jeg har lige opdateret min fil med en frisk kopi fra siden, så må vi se hvordan den opføre sig. Jeg har kaldt min for # v0.0.6 29-01-2024 17:19

Andre sensore logger fint nok med data, men der kan være sket en fejl under en opdatering jeg husker ikke lige hvordan og hvornår jeg har opdateret noget andet end den jeg lige har overskrevet ved jeg ikke, den jeg lige har overskrevet har jeg noteret til # v0.0.5 28-01-2024

De første opdateringer jeg lavede fulgte jeg dine versioner i opslaget på facebook

JensenNick commented 5 months ago

jeg har lige opdateret min fil med en frisk kopi fra siden, så må vi se hvordan den opføre sig. Jeg har kaldt min for # v0.0.6 29-01-2024 17:19

@JacobsenKim Jeg har opdateret package filen. Tror jeg har fået det fikset nu. Se issue som der er linket til lige herover. Test kører på en helt fresh install af HA og ser rigtig ud. Men det jeg har fundet forklarer ikke rigtigt dine grafer - eller jeg kan i hvert fald ikke gennemskue hvad der er sket hos dig kl. 06 den 26.

Jeg skal nok begynde at give versions numre, men vi er nødt til at komme over disse start vanskeligheder, der desværre overrasker mig lidt. Jeg har trods alt haft det kørende en måned på min maskine. Så kom der godt nok en rettelse med afladning af batteri til net og så burde jeg nok have haft en testmaskine sat op noget tidligere.

JacobsenKim commented 5 months ago

Jeg aner ikke hvad der er sket men kan se den omkring kl 10 (hvor jeg sov) er begyndt at bevæge sig lidt men klart se en forskel efter jeg hentede en frisk kopi og indsatte el pris sensor samt invertere correction.

Jeg kan have været inde på filen og måske trykket på forkert keyboard og skrevet noget jeg ikke har set også senere gemt og igen senere genstartet ha, det er også lige meget, det ser ud til at være liv nu.

Skærmbillede fra 2024-01-29 21-48-02

Ja kan godt se du har lavet om på sensor navne, det giver også mere mening at kalde sensor 2 for 2 end (), jeg bøvlede noget rundt i det da jeg kiggede på den sensor.

ang. test maskine så kan invertere ikke lide at tale med 2 ha maskiner på en gang. her har jeg haft en VM kørende fra ca. 17 til ca. 13

Skærmbillede fra 2024-01-29 14-28-12

Jeg må igang igen med at opdatere :-D Tænker at ligge

Efficiency Corrected Power Input Sensor #8

ind i en separat fil så jeg ikke skal bøvle med at kopiere den over hver gang, men ved ikke om det vil virke på den måde.

Super med v-nr. :-D

JensenNick commented 5 months ago

@JacobsenKim Jeg har med de seneste revisioner rettet på ovenstående og lukker derfor issue. Tak for indspark.

2024-01-30_10-36-58