muppet3000 / homeassistant-growatt_server_api

Home Assistant Integration for Growatt - Upstream repo for the growatt_server integration that is part of the Core Home Assistant repository
MIT License
54 stars 7 forks source link

ENHANCEMENT - Add extra values (like those available for Hybrid systems) for TLX (MIN/MIC/MOD/NEO) systems #12

Open muppet3000 opened 1 year ago

muppet3000 commented 1 year ago

Describe the bug Continuation of the work that was started in this ticket: https://github.com/home-assistant/core/issues/81470 Potential overlap with: https://github.com/home-assistant/core/issues/82090 (although I think this was fixed in HA 2022.12.7)

Growatt Device Type

Device(s): TLX aka MIN/MIC/MOD

Home Assistant information

Additional context

Would be good to add these values at the same time as fixing: https://github.com/muppet3000/homeassistant-growatt_server_api/issues/4 NOTE - To add these values it requires new API endpoints to be added to the upstream PyPi library.

muppet3000 commented 1 year ago

@BigFam194 @bartpaeleman @Saentist @charlie063 @danant96 @HilbertusO - This is a continuation of the work I started here: https://github.com/home-assistant/core/issues/81470 Specifically with reference to this comment: https://github.com/home-assistant/core/issues/81470#issuecomment-1355007106

@sidequest-tomas @j1and1 - This is a continuation of the bug you logged here: https://github.com/home-assistant/core/issues/82090 so that I can track it in the new home for the integration.

NOTE - I have all the information I need to progress this, I just wanted to let you all know that I'm tracking it over here where I'm primarily maintaining the integration going forward (rather than in the 'core' repo). I will still integrate changes over there, but they will be in slower time.

muppet3000 commented 1 year ago

@sidequest-tomas - I've just seen this comment here: https://github.com/home-assistant/core/issues/81470#issuecomment-1366754550 If you still believe this to be an issue could you please log it as a new bug in this repo as I think I can fix that one separately.

muppet3000 commented 1 year ago

Note for myself: There is a comment here: https://github.com/home-assistant/core/issues/82090#issuecomment-1404024891 suggesting that some of the incrementing totals should be added as TOTAL_INCREASING, I need to validate this and update any that are missing.

horstefan commented 1 year ago

Hi

I own a Growatt MOD 5000 TL3-X 3-fase which is recognised by the integration but device values are not populated (zero) Im not sure if this is already in your scope. if not please let me know, Im willing to share login credentials with you.

The plant global sensors do work as expected.

AC frequency 0 Hz Energy today 0 kWh Input 1 Amperage 0 A Input 1 voltage 0 V Input 1 Wattage 0 W Input 2 Amperage 0 A Input 2 voltage 0 V Input 2 Wattage 0 W Input 3 Amperage 0 A Input 3 voltage 0 V Input 3 Wattage 0 W Intelligent Power Management temperature 0 °C Internal wattage 0 W Lifetime energy output 0 kWh Output power 0 W Reactive amperage 0 A Reactive voltage 0 V Reactive wattage 0 W Temperature 0 °C

muppet3000 commented 1 year ago

Hi

I own a Growatt MOD 5000 TL3-X 3-fase which is recognised by the integration but device values are not populated (zero) Im not sure if this is already in your scope. if not please let me know, Im willing to share login credentials with you.

The plant global sensors do work as expected.

AC frequency 0 Hz Energy today 0 kWh Input 1 Amperage 0 A Input 1 voltage 0 V Input 1 Wattage 0 W Input 2 Amperage 0 A Input 2 voltage 0 V Input 2 Wattage 0 W Input 3 Amperage 0 A Input 3 voltage 0 V Input 3 Wattage 0 W Intelligent Power Management temperature 0 °C Internal wattage 0 W Lifetime energy output 0 kWh Output power 0 W Reactive amperage 0 A Reactive voltage 0 V Reactive wattage 0 W Temperature 0 °C

This is a weird one, could you please log a new ticket as a bug, using the bug template. This ticket is for extending the existing functionality for the TLX system, which should already be working. So having a new ticket to track your issue in isolation would be best.

Maxpako commented 1 year ago

Hello, Very good integration, but I'm missing the data about imported electricity from my MIN 3600TL-XH inverter connected to a SDM meter. But I can see all this info in growatt server. Any way that data can be added to the integration?

Edit: I'm missing load_consumption (actual, not lifetime) and imported from grid (lifetime and actual). Those values are shown in growatt server website.

muppet3000 commented 1 year ago

Given the fact that the Growatt API is increasingly unreliable (see #15) I have slowly winding down support for this repo and instead directing people towards trying Grott instead: https://github.com/muppet3000/homeassistant-grott

I'm happy to help people out migrating over to the new method, it's far more reliable and I'm focussing all my efforts into making it the best possible option for everyone (including feeding back changes to the main developer)

sidequest-tomas commented 1 year ago

I totally understand, and i will try out Grott , but is the process reversible? So you can change back to Growatts unreliable API if you want later?

The instructions on Grott is quite good, but not fool-proof :D