homeassistant-growatt_server_api icon indicating copy to clipboard operation
homeassistant-growatt_server_api copied to clipboard

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

Open muppet3000 opened this issue 2 years ago • 8 comments

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

  • Home Assistant version: N/A
  • Growatt Server API Integration version: N/A

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 avatar Jan 24 '23 21:01 muppet3000

@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 avatar Jan 24 '23 21:01 muppet3000

@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 avatar Jan 24 '23 21:01 muppet3000

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.

muppet3000 avatar Jan 25 '23 21:01 muppet3000

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

horstefan avatar Jan 29 '23 21:01 horstefan

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.

muppet3000 avatar Jan 30 '23 19:01 muppet3000

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.

Maxpako avatar Mar 21 '23 10:03 Maxpako

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)

muppet3000 avatar Jun 20 '23 19:06 muppet3000

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

sidequest-tomas avatar Jun 21 '23 07:06 sidequest-tomas