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

changed time offset open-meteo.yaml #19507

Merged
merged 2 commits into from
Mar 7, 2025
Merged

changed time offset open-meteo.yaml #19507

merged 2 commits into from
Mar 7, 2025

Conversation

thecem
Copy link
Contributor

@thecem thecem commented Mar 6, 2025

changed th time offset for value calculation.

@andig
Copy link
Member

andig commented Mar 7, 2025

Das sieht nach einer ganz schlechten Idee aus, es sei denn du hättest das mal in einer anderen Zeitzone getestet?

@andig
Copy link
Member

andig commented Mar 7, 2025

Lässt sich das Api vllt. zu UTC überreden?

@andig andig added the tariffs Specific tariff support label Mar 7, 2025
@thecem
Copy link
Contributor Author

thecem commented Mar 7, 2025

Die Zeit Daten von open-meteo kommen in geo coordinator Zeit, somit ist eine Umrechnung nur nötig, wenn ich in DE eine anderen Zeitzonen PV monitoren will. Sprich gebe ich die coordinates von düsseldorf an ist es mez/mesz, wenn ich die coordinates von Johannisburg angebe ist es JNB.
Allerdings werden die Zeiten von dem script richtig übergeben, aber von evcc um 1h verschoben. Wenn Du hier helfen kannst gerne.
Auch die zeit der API lässt sich auf GMT stellen, ändert aber nichts an dem verschub (getestet), gerne kannst Du hier unterstützen. Was dass nun macht ist die Berechnung der Werte um einen index nach vorne zu ziehen.

@andig
Copy link
Member

andig commented Mar 7, 2025

Sprich gebe ich die coordinates von düsseldorf an ist es mez/mesz, wenn ich die coordinates von Johannisburg angebe ist es JNB.

Dann wird der Parser für JNB auf die Nase fallen wenn evcc auf MEZ läuft.

Auch die zeit der API lässt sich auf GMT stellen,

Dann wäre das besser.

ändert aber nichts an dem verschub (getestet)

Das wäre dann ein Bug im API. Das erscheint nicht ganz wahrscheinlich- wie kann man GMT falsch hin kriegen?

@thecem
Copy link
Contributor Author

thecem commented Mar 7, 2025

Ich habe das jetzt noch mal in 6 Zeitzonen getestet und die Anzeige in evcc passt mit dieser Lösung.
Wo ist das problem?

Wenn ich die Zeit auf JNB umstelle (evcc Host und PC) dann wird die kurve mit dem offset richtig angezeigt und zwar zu dem offset von jnb zu den tatsächlichen werten der PV geocoordinaten.
Wenn ich die zeit auf asia/tokyo in der api umstelle und der evcc host und pc läuft auf MSZ dann wird die kurve ebenfalls zu den geo coordinaten der PV richtig angezeigt.
Stelle ich die geo coordinaten auf Tokyo wird der Ertrag im pc mit gmt+9 angezeigt also auch korrekt.
Klar kann man auf GMT umstellen, löst aber nicht das Problem.

Wie gesagt die Änderung ändert nichts an der Zeitzone und dessen Berechnung, das bleibt wie es ist, ich schiebe nur den Wert um genau ein index nach vorne.

@andig andig merged commit 6367ad6 into evcc-io:master Mar 7, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tariffs Specific tariff support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants