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

Add total yield to Kostal and Hoymiles OpenDTU template #15259

Merged
merged 20 commits into from
Aug 8, 2024

Conversation

xerion3800
Copy link
Contributor

No description provided.

@andig andig added the devices Specific device support label Aug 5, 2024
@andig andig requested a review from premultiply August 5, 2024 19:00
@xerion3800
Copy link
Contributor Author

xerion3800 commented Aug 7, 2024

@andig habe jetzt noch mal die Änderungen für den Bereich "pv" vorgenommen. Kannst du bitte noch einmal drüber schauen.
kleine OT Frage. ich habe die Änderungen ja hauptsächlich gemacht, da ich gemerkt habe, das die Tageserträge der PV in der InfluxDB nicht passend waren. Jetzt werden die Werte von energy angezeigt aber das value von dem measurement pvDailyEnergy bildet immer noch nicht die richtigen Summen ab. Muss ich dort noch mehr tun außer in dem template?

@andig
Copy link
Member

andig commented Aug 7, 2024

@naltatis ?

@naltatis
Copy link
Member

naltatis commented Aug 7, 2024

Der UI Step schlägt hier fehl, weil am die Leerzeile am Ende der yaml-Dateien [1][2] entfernt wurden.

@naltatis
Copy link
Member

naltatis commented Aug 7, 2024

@xerion3800 Zur measurement pvDailyEnergy Frage kann ich nichts sagen. Das muss dann ja eine Logik in Influx oder so sein.

@xerion3800
Copy link
Contributor Author

Der UI Step schlägt hier fehl, weil am die Leerzeile am Ende der yaml-Dateien [1][2] entfernt wurden.

Habe das korrigiert und neu comittet. Sorry dafür bin noch am üben...

@xerion3800
Copy link
Contributor Author

Der UI Step schlägt hier fehl, weil am die Leerzeile am Ende der yaml-Dateien [1][2] entfernt wurden.

Ich habe die leeren Zeilen am Ende eingefügt aber der Build schlägt immer noch fehl., Kannst du bitt einmal schauen. @naltatis

Copy link
Member

@premultiply premultiply left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Zumindest was die Formatierung usw. angeht sollte es jetzt stimmen.
Ob die Datenzuordnung stimmt kann ich mangels Gerät natürlich nicht bewerten.

@andig
Copy link
Member

andig commented Aug 8, 2024

Kleiner Wunsch noch: bitte Energy immer nach Power im Template. Power ist muss, Energy ist kann.

@deadrabbit87
Copy link
Contributor

deadrabbit87 commented Aug 8, 2024

Zumindest was die Formatierung usw. angeht sollte es jetzt stimmen.
Ob die Datenzuordnung stimmt kann ich mangels Gerät natürlich nicht bewerten.

Ich bin aktuell im Urlaub und die Datenverbindung ist hier echt mies, aber ich könnte ggf gerne was an einem Kostal Plenticore mit BYD testen falls noch notwendig....

@andig
Copy link
Member

andig commented Aug 8, 2024

Letzter Wunsch: könntest Du bei templates/definition/meter/kostal-piko-pv.yaml bitte auch noch den type: custom und das power Property je if doppeln? Sonst kommt das zusätzliche Energy am Ende arg überraschend.
Und ist es richtig, dass Du energy jeweils bei PV ergänzt hast, aber nicht bei Grid?

@xerion3800
Copy link
Contributor Author

Letzter Wunsch: könntest Du bei templates/definition/meter/kostal-piko-pv.yaml bitte auch noch den type: custom und das power Property je if doppeln? Sonst kommt das zusätzliche Energy am Ende arg überraschend.

Klar baue ich ein

Und ist es richtig, dass Du energy jeweils bei PV ergänzt hast, aber nicht bei Grid?

Da ich nicht wirklich die Auswirkungen kenne was genau bei Grid benötigt wird und es es auch nicht testen kann habe ich es vorsichtshalber raus gelassen.

@andig andig merged commit 13dca38 into evcc-io:master Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants