muppet3000 / homeassistant-grott

Home Assistant Integration for Grott - MQTT
MIT License
65 stars 14 forks source link

Missing values eacDischarge etc, SPF5000 es inverter #57

Open yvesdemoerloose opened 8 months ago

yvesdemoerloose commented 8 months ago

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 commented 8 months ago

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

muppet3000 commented 8 months ago

@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.

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.

yvesdemoerloose commented 8 months ago

@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

muppet3000 commented 8 months ago

@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.

yvesdemoerloose commented 8 months ago

Sorry for the confusion,

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.

muppet3000 commented 8 months ago

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

yvesdemoerloose commented 8 months ago

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

muppet3000 commented 8 months ago

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

github-daniel-stoian commented 7 months ago

@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:

image image image image image
muppet3000 commented 7 months ago

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.

github-daniel-stoian commented 7 months ago

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 commented 7 months ago

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 commented 6 months ago

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

Thank you.

muppet3000 commented 6 months ago

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.