-
-
Notifications
You must be signed in to change notification settings - Fork 224
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
Fix HA autodiscovery and add total entities #630
Conversation
please check with latest version |
DynamicJsonDocument doc(128) to 256 (else mf=nul) Why not create a DTU entity grouping the totals, rather than repeating them for each inverter? |
I think it was too late for me, thank you for your review!
It makes no sense to publish the totals more than once.
|
Now, Yieldday and Yieldtotal from the total topic have no exp_after flag in autodiscovery. |
…sh YD & YT on CH0 only when iv is producing to avoids 0 at restart.
@antibill51 maybe you take a look at the fix for #635 i think such a chunked request would solve the HA auto discovery problem too. Thanks! |
@stefan123t I am interested in this kind of project to deepen my basics in code development. To be honest, I just discovered the term "prometheus metrics"... I'm not sure of the result, but I'll try to move on it as soon as possible. It's not easy to understand the whole of a project like yours, I took a while to make the previous few changes, but I'm someone who doesn't like to use a v0.5.41 when v0.5.7x exists ;) |
Fix HA autodiscovery.
Add items from "total" topic into HA autodiscovery.
Optimizations are probably needed.