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

Missing values eacDischarge etc, SPF5000 es inverter

Open yvesdemoerloose opened this issue 2 years ago • 14 comments

Hi, I'm using the grott addon and tried also your grott integration I get this data from the grott logs image

I get this in the mqtt device: image image

From your integration I get: image

EDIT: I think the 6th sensor in the mqtt device is wrongly named/interpreted. This is the grid import or consumption today (not much sun, so on grid all day already, batteries charging only from sun) Thanks!

yvesdemoerloose avatar Oct 19 '23 12:10 yvesdemoerloose

Ok found the problem, the AC_discharge values were wrongly labeled as coming from the battery. Changed the entities manually, all good now.

yvesdemoerloose avatar Oct 19 '23 16:10 yvesdemoerloose

@yvesdemoerloose please can you help me so that I can fix it for everyone?

Can you tell me what sensors were incorrect and what they should be instead? I need to know what the name of the stat is in the grott output and what it is in 'human readable' terms and also what unit of measurement they're supposed to be e.g. Watt. Bonus points for doing it as a tick list like below so I can tick them off as I fix them. e.g.

  • [ ] ACDischarWatt -> "The correct name for the sensor" (Watt)

Once you've got me those I can hunt down what's incorrect and get them all fixed really easily. It's really tricky to implement these things without having the system yourself so I rely on feedback like this to populate it correctly.

muppet3000 avatar Oct 20 '23 05:10 muppet3000

@yvesdemoerloose please can you help me so that I can fix it for everyone?

Can you tell me what sensors were incorrect and what they should be instead? I need to know what the name of the stat is in the grott output and what it is in 'human readable' terms and also what unit of measurement they're supposed to be e.g. Watt. Bonus points for doing it as a tick list like below so I can tick them off as I fix them. e.g.

  • [ ] ACDischarWatt -> "The correct name for the sensor" (Watt)

Of course Chris,

ACDischarWatt -> grid_discharge_power

What was wrong here (SPF model): eacDischargeTotal= had entity: lifetime_battery_discharged eacDischargeToday= had entity: battery_dischaged_today (the typo in discharged is present, so should be easy to find)

EDIT: To be sure it's correct noted, the wrong sensors in my case were named as coming from the battery, while their source was grid.

Hope this helps, if any other questions, just ask, Thx for your work Yves

yvesdemoerloose avatar Oct 20 '23 16:10 yvesdemoerloose

@yvesdemoerloose please can you help me so that I can fix it for everyone?

Can you tell me what sensors were incorrect and what they should be instead? I need to know what the name of the stat is in the grott output and what it is in 'human readable' terms and also what unit of measurement they're supposed to be e.g. Watt. Bonus points for doing it as a tick list like below so I can tick them off as I fix them. e.g.

  • [ ] ACDischarWatt -> "The correct name for the sensor" (Watt)

Of course Chris,

ACDischarWatt -> grid_discharge_power

What was wrong here (SPF model): eacDischargeTotal= had entity: lifetime_battery_discharged eacDischargeToday= had entity: battery_dischaged_today (the typo in discharged is present, so should be easy to find)

EDIT: To be sure it's correct noted, the wrong sensors in my case were named as coming from the battery, while their source was grid.

Hope this helps, if any other questions, just ask, Thx for your work Yves

I'm sorry, I'm still a little confused, please could you provide a bullet list in the format I suggested, I just need the Grott name and then the human name and whatever format it's in. You've told me what entity things had, but not what they should be.

muppet3000 avatar Oct 20 '23 18:10 muppet3000

Sorry for the confusion,

  • eacDischargeTotal -> Import from grid lifetime (Kwh)
  • eacDischargeToday -> Import from grid today (Kwh)

Kind regards Yves

PS In your integration I only have 4 sensors and 4 diagnostic entries, in the MQTT device made by Grott addon, I got 47 entities and 5 diagnostic entries. See screenshots above.

yvesdemoerloose avatar Oct 21 '23 07:10 yvesdemoerloose

ahhh, I've just realised what's happening here - I thought I'd already implemented these - and I have :D However, they're not in a formal release yet. Please could you install the 'main' version of the integration instead of the latest release. Everything is currently on there for testing, please give me feedback for anything that is incorrect.

If you're unsure on how to install it I made some notes here for someone else: https://github.com/muppet3000/homeassistant-grott/issues/49#issuecomment-1756167643

muppet3000 avatar Oct 22 '23 06:10 muppet3000

Please could you install the 'main' version of the integration

hi,

Yep, after installing the main version through hacs, I got all the necessary entities in your integration also. That indeed fixed it :-)

image

yvesdemoerloose avatar Oct 22 '23 16:10 yvesdemoerloose

It's probably about time I did another release then!

muppet3000 avatar Nov 01 '23 20:11 muppet3000

@muppet3000 @yvesdemoerloose Are you sure you got "all the necessary entities" ?

I have a similar setup : -Growatt SPF 5000ES -same data from grott logs -same Grott integration 'main' installed via Hacs -same 1device in Grott integration -same 39entities in Grott integration

But "Grid Input Power" or "Grid Apparent Input Power" are not in the 39 entities.

Basically I have no sensor for:

  • Grid Import Power (to see how much power the inverter is taking from Grid)
  • Daily Load Consumption (to see how much energy i've consumed Today)
image image image image image

github-daniel-stoian avatar Nov 29 '23 10:11 github-daniel-stoian

Which ones from the screengrab you posted off the grott logs are the values you're looking for, if you tell me those then I can map them through.

muppet3000 avatar Nov 29 '23 21:11 muppet3000

Here they are :

AC_InWatt = Grid Input power AC_InVA = Grid Input apparent power Load Consumption Today = eopDischrToday Load Consumption Total = eopDischrTotal

Those last two (Load Consumption) I cannot find them in Grott logs. They ware present in the old growatt server https://github.com/indykoning/home-assistant-growatt-server/blob/master/sensor.py#L163

Thank you.

github-daniel-stoian avatar Nov 30 '23 06:11 github-daniel-stoian

I've tried to raise a PR for adding AC_InWatt = Grid Input power sensor. But I don't have the permissions. Thank you.

{ "name": "Grid Input power", "device_class": SensorDeviceClass.POWER, "unit_of_measurement": UnitOfPower.WATT, "state_class": SensorStateClass.MEASUREMENT, "icon": "mdi:battery", "func": lambda js: js['values']["AC_InWatt"], "divider": 10, "unique_name": "mqtt_136", }

github-daniel-stoian avatar Dec 04 '23 07:12 github-daniel-stoian

@muppet3000 How long will this take ? Should I raise another issue for it ?

Thank you.

github-daniel-stoian avatar Dec 06 '23 12:12 github-daniel-stoian

Sorry @github-daniel-stoian you should be able to raise a PR without any issues. Do you have a fork of the codebase with the change in it so I can cherry-pick? Otherwise I'll try to get this in over the weekend.

muppet3000 avatar Dec 06 '23 20:12 muppet3000