weltmeyer / ha_sonnenbatterie

Homeassistant integration to show many stats of Sonnenbatterie
GNU General Public License v3.0
51 stars 24 forks source link

PPV2 values? #19

Closed jepperc closed 1 year ago

jepperc commented 1 year ago

Can the api this integration calls get the PPV2 values? After losing access to the installer section(Sonnen upgrade) i can't get readings for the secondary ppv in the ui. But it looks like the api here returns it, just doesn't populate sensor values?

weltmeyer commented 1 year ago

Do you know where in the api the value is returned?

jepperc commented 1 year ago

It could just be called PPV2 here? https://github.com/weltmeyer/ha_sonnenbatterie/blob/78d8a6a602e35a66574c510507fa68ade7fbc369/custom_components/sonnenbatterie/sensor.py#L291 In the old interface (http://sonnenbattery:8080) you could login with installer and see this exact value. Since this is the old API i would guess the same value would be there.

xlibbyx commented 1 year ago

I would also be interested in the PPV2 value. Is it possible to add myself?

KlavsKlavsen commented 1 year ago

@xlibbyx Here's a discussion on how someone improved upon this - to use apiV2 Hopefully this could be integrated into this plugin - to make it much easier to use. https://community.home-assistant.io/t/sonnenbatterie-with-apiv2-webhook/264907

I just got my battery setup and have yet to solve this part myself. Weltmeyer's github repo was fortunately easy to add as a custom integration and it showed up.. these improvements - not so much.. so we'll see if I figure it out (if I do - I'll certainly try to figure out how to improve this module and contribute PR)

jepperc commented 1 year ago

How would that help anything? It doesn't have PPV2 values 😬

tor. 15. dec. 2022 16.29 skrev Klavs Klavsen @.***>:

Here's a discussion on how someone improved upon this - to use apiV2 Hopefully this could be integrated into this plugin - to make it much easier to use.

https://community.home-assistant.io/t/sonnenbatterie-with-apiv2-webhook/264907

— Reply to this email directly, view it on GitHub https://github.com/weltmeyer/ha_sonnenbatterie/issues/19#issuecomment-1353271134, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAA7FVNNZULXLOSHL4EJ62LWNM2N3ANCNFSM6AAAAAARN2ZLW4 . You are receiving this because you authored the thread.Message ID: @.***>

KlavsKlavsen commented 1 year ago

the ppv2 values aren't in the new API output on your system? if not - then I guess sonnen decided you're not supposed to know this - or its a bug that should be reported to Sonnen (everything should AFAIK be accessible via the apiV2 ?)

weltmeyer commented 1 year ago

as i dont have an inverter, i cant check for these ppv values. i blindly added ppv2 now. can you try latest master?

jepperc commented 1 year ago

the ppv2 values aren't in the new API output on your system? if not - then I guess sonnen decided you're not supposed to know this - or its a bug that should be reported to Sonnen (everything should AFAIK be accessible via the apiV2 ?)

No.. and it's not in the screenshots in the link you provided either. There is not PPV1 either.. only Production_W which is the combined.

jepperc commented 1 year ago

as i dont have an inverter, i cant check for these ppv values. i blindly added ppv2 now. can you try latest master?

Will try tonight when I get my new HA server up and running :)

xlibbyx commented 1 year ago

as i dont have an inverter, i cant check for these ppv values. i blindly added ppv2 now. can you try latest master?

Thanks for this. I have updated and added PPV2 to my frontend so I will report back when the sun is up tomorrow.

xlibbyx commented 1 year ago

ppv

Looks like it worked!

weltmeyer commented 1 year ago

nice. gonna close this ticket.

jepperc commented 1 year ago

Works great :) Thank you!