Skip to content

Conversation

@cshagen
Copy link
Collaborator

@cshagen cshagen commented Sep 13, 2025

  • Preis-Modul für EKZ
  • Preis-Modul für Group E

@cshagen cshagen changed the title Electricity tariffs for Switzerland Electricity tariffs (EKZ, Groupe E) Sep 14, 2025
@LKuemmel LKuemmel added this to the 2.1.9 milestone Sep 23, 2025
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]]:
Copy link
Contributor

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]]:
Copy link
Contributor

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.

Copy link
Collaborator Author

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.

Copy link
Contributor

@tpd-opitz tpd-opitz Oct 23, 2025

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...

Copy link
Collaborator Author

@cshagen cshagen Oct 23, 2025

Choose a reason for hiding this comment

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

Es wäre gut zu wissen, ob die beiden Changes gemeinsam gemerged werden. Im Moment sieht es so aus, als wäre dieser Change für 2.1.9alpha2 eingeplant. #2801 aber noch nicht. D.h. ich mache dann nochmal ein Update sobald #2801 drin ist.

Copy link
Contributor

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.

Copy link
Contributor

@tpd-opitz tpd-opitz Oct 24, 2025

Choose a reason for hiding this comment

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

@LKuemmel: Ist zwar nur 'ne Kleinigkeit, aber dieseer PR gehört prinzipiell auch mit dazu: #807

Copy link
Collaborator Author

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?

@cshagen
Copy link
Collaborator Author

cshagen commented Oct 24, 2025

Die Umstellung auf 15 Minuten ist implementiert.

@tpd-opitz
Copy link
Contributor

ein paar UnitTests wären noch ganz nett... ;o)

@cshagen
Copy link
Collaborator Author

cshagen commented Oct 27, 2025

ein paar UnitTests wären noch ganz nett... ;o)

Gibe es Beispiele?

@tpd-opitz
Copy link
Contributor

@cshagen
Copy link
Collaborator Author

cshagen commented Oct 27, 2025

Ok, das muss ich mir bei Gelegenheit mal in Ruhe anschauen.

@cshagen
Copy link
Collaborator Author

cshagen commented Oct 27, 2025

PR 821 im openwb-ui-settings-repo ergänzt diesen PR

@tpd-opitz
Copy link
Contributor

PR 821 im openwb-ui-settings-repo ergänzt diesen PR

#821

@cshagen
Copy link
Collaborator Author

cshagen commented Oct 27, 2025

PR 821 im openwb-ui-settings-repo ergänzt diesen PR

#821

Nö, nicht der, sondern der: openWB/openwb-ui-settings#821

@LKuemmel LKuemmel merged commit 6ad2022 into openWB:master Nov 4, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants