-
Notifications
You must be signed in to change notification settings - Fork 42
Goodwe Homekit #59
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
Comments
Yes indeed. For the statistics to work, you need a separate sensor with the correct |
The README mentions how to create your own additional sensors from the state of the main sensor exposed by this integration. However, isn't it a better idea to expose all values in the state as separate sensor by default? This would allow the integration in add the proper An alternative Goodwe integration (which works via UDP, see: https://github.com/mletenay/home-assistant-goodwe-inverter) does exactly this. |
on core-2022.5.4 Firstly- great work @TimSoethout . I was unable to use any of the local integrations with my Goodwe inverter because it upset the SEMS uploads (even if I set it to poll every 3 or 5 mins!) and none seem compatible with the GoodWe homekit. I have a GoodWe inverter (GW10KAU-DT), plus GoodWe homekit (for three-phase power).
both in kWh of what should be only increasing values of my total grid consumption (eg buying), and grid export (selling) respectively. This affects the HA Energy page. |
I've posted this elsewhere, but just in case it helps. For the issue with the dip in values, I'm now trying to change the all time totals to total_increasing and see if that avoids the issue.
|
I hope it fixes your issue! I didn't think the PR would be merged so quickly or really at all so I didn't get a chance to mention that when switching from Totals to Charts data you'll see a new kind of spike in your graph - but it's only once off. Luckily most people with this issue have broken graphs already and the next day should have a fixed graph. For my own sake I'm also writing a tool to repopulate all available sems data to give me clean historical graphs. |
Thanks so much @Denifia everything is working! Yes I had the one-off spike at midnight (reverse to the usual one) and it is good since. |
@gemmaker glad the fix worked for you! I'll share my cleanup tool when it's working for me but I don't get a lot of time to work on it so it could be a few weeks away. |
@gemmaker I've put up my first cut of a Sems HomeKit importer at https://github.com/Denifia/goodwe-sems-history-home-assistant but in some weird turn of fate that has a bug where recalculated days that were not imported have the daily data spike. I've run out of time to work on it but hopefully someone else in the community will be able to contribute. |
Great work on the addition of the Homekit sensor.
The data from this sensor in HA I have worked out from the data and comparing to my SEMS screen is the "consumption".
This sensor however is not available in the drop down box when trying to configure HA Energy.
This is also the same for trying to add any other Goodwe sensor to the HA Energy, they are not available to select to "add solar production", "add consumption" or "add return".
The Goodwe homekit has 2 CT's one reads solar production and the other reads the "feed from" or "return to" the grid.
From here consumption should just be a calculation of solar production, plus minus feed to or from the grid.
This is obviously how the SEMS portal calculates it.
I think we should be seeing 3 Homekit sensors. (From/to grid, consumption, solar production)
Not being able to add the Homekit sensor to HA is aparrantly due to the setup/coding of the sensor;
This is what HA says:
You’re configuring a statistic but you couldn’t find your source in the dropdown? That’s caused by a bug in the integration providing the entity. Integrations need to configure their entities correctly so Home Assistant knows that we need to track statistics for it and how.
Open an issue with the author of the integration and link them to https://developers.home-assistant.io/docs/core/entity/sensor#long-term-statistics.
The text was updated successfully, but these errors were encountered: