Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
muppet3000 opened this issue Jan 24, 2023 · 8 comments
Labels
enhancement New feature or request Growatt:TLX

Comments

@muppet3000
Copy link
Owner

Describe the bug
Continuation of the work that was started in this ticket: home-assistant/core#81470
Potential overlap with: home-assistant/core#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: #4
NOTE - To add these values it requires new API endpoints to be added to the upstream PyPi library.

@muppet3000 muppet3000 added bug Something isn't working triage Needs to be triaged to determine if it's an issue labels Jan 24, 2023
@muppet3000
Copy link
Owner Author

@BigFam194 @bartpaeleman @Saentist @charlie063 @danant96 @HilbertusO - This is a continuation of the work I started here: home-assistant/core#81470
Specifically with reference to this comment: home-assistant/core#81470 (comment)

@sidequest-tomas @j1and1 - This is a continuation of the bug you logged here: home-assistant/core#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
Copy link
Owner Author

@sidequest-tomas - I've just seen this comment here: home-assistant/core#81470 (comment)
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 muppet3000 added enhancement New feature or request Growatt:TLX and removed bug Something isn't working labels Jan 24, 2023
@muppet3000 muppet3000 changed the title BUG - Add extra values (like those available for Hybrid systems) for TLX (MIN/MIC/MOD/NEO) systems ENHANCEMENT - Add extra values (like those available for Hybrid systems) for TLX (MIN/MIC/MOD/NEO) systems Jan 24, 2023
@muppet3000 muppet3000 removed the triage Needs to be triaged to determine if it's an issue label Jan 24, 2023
@muppet3000
Copy link
Owner Author

Note for myself: There is a comment here: home-assistant/core#82090 (comment) 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
Copy link

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
Copy link
Owner Author

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
Copy link

Maxpako commented Mar 21, 2023

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
Copy link
Owner Author

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
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Growatt:TLX
Projects
None yet
Development

No branches or pull requests

4 participants