-
Notifications
You must be signed in to change notification settings - Fork 107
Electricity tariffs (EKZ, Groupe E) #2757
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
Conversation
cshagen
commented
Sep 13, 2025
- Preis-Modul für EKZ
- Preis-Modul für Group E
Co-authored-by: LKuemmel <76958050+LKuemmel@users.noreply.github.com>
|
|
||
|
|
||
| # Aggregate 15min prices to hourly prices by taking the maximum price in each hour | ||
| def aggregatePrices(quarterlyPrices) -> list[tuple[str, float]]: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unterstützung von 15-Minuten Preis-Intervallen ist schon in Vorbereitung, daher würde ich auf diese Optimierung verzichten.
|
|
||
|
|
||
| # Aggregate 15min prices to hourly prices by taking the maximum price in each hour | ||
| def aggregatePrices(quarterlyPrices) -> list[tuple[str, float]]: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unterstützung von 15-Minuten Preis-Intervallen ist schon in Vorbereitung, daher würde ich auf diese Optimierung verzichten.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, ich kann das gerne rausnehmen, solange es keine Probleme mit der aktuellen Implementierung gibt.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Die Aktuelle Implementierung hat sicher Probleme, weil sie bei der Niedrigpreis-Suche alle Zeitslots benutzt, bei der Entscheidung ob geladen werden soll aber immer den ersten der aktuellen Stunde, so dass im Zweifel nicht geladen wird.
Wenn Du das drin lässt musst Du es halt dann noch mal anfassen, wenn #2801 gemerged ist...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich habe übersehen, dass EKZ 15-Minuten Intervalle benutzt. Ich habe es in Alpha 3 verschoben, da soll das Merging nächste Woche beginnen. Dann merge ich das zusammen.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ich habe übersehen, dass EKZ 15-Minuten Intervalle benutzt. Ich habe es in Alpha 3 verschoben, da soll das Merging nächste Woche beginnen. Dann merge ich das zusammen.
Soll ich dann die Aggregation auf 1h-Intervalle noch rausnehmen?
|
Die Umstellung auf 15 Minuten ist implementiert. |
|
ein paar UnitTests wären noch ganz nett... ;o) |
Gibe es Beispiele? |
|
|
Ok, das muss ich mir bei Gelegenheit mal in Ruhe anschauen. |
|
PR 821 im openwb-ui-settings-repo ergänzt diesen PR |
|
Nö, nicht der, sondern der: openWB/openwb-ui-settings#821 |