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

Tariff: fill empty slots with zero #18788

Closed
wants to merge 1 commit into from
Closed

Tariff: fill empty slots with zero #18788

wants to merge 1 commit into from

Conversation

andig
Copy link
Member

@andig andig commented Feb 14, 2025

Fix #18773

@naltatis damit gibts bei Lücken- z.B. zwischen letzter und aktueller Abfrage- aber keine ? mehr sondern jetzt Nullwerte. Bin gar nicht sicher, ob das eine gute Idee ist? Zumindest der Planner heute kann ja auch mit fehlenden Daten umgehen.

@andig andig added the tariffs Specific tariff support label Feb 14, 2025
@andig andig requested a review from naltatis February 14, 2025 08:22
@naltatis
Copy link
Member

zwischen letzter und aktueller Abfrage- aber keine ? mehr sondern jetzt Nullwerte.

Was hältst du davon Price optional zu machen? Bei Leistung/Energie ist ein Default von 0 ok, bei Strompreisen und CO2 wäre 0ct ja etwas was starke Auswirkungen auf den Algorithmus hätte.

@andig
Copy link
Member Author

andig commented Feb 14, 2025

Ungern weil das an vielen Stellen im Code auftaucht. All das wg. eines einzigen Charts. Finden wir sonst doch eine Lösung im Chart?

@naltatis
Copy link
Member

Und es nur bei Solar (typabhängig) auffüllen?

@andig andig marked this pull request as draft February 14, 2025 11:51
@andig
Copy link
Member Author

andig commented Feb 14, 2025

Das wäre zumindest an der Stelle nicht möglich. Wie gross ist das Problem denn wirklich- kann die Chart library nicht damit umgehen oder wo ist unser Klemmer?

@naltatis
Copy link
Member

Ist nicht nur die Chart-Library. Ich checke aktuell bspw. ob die Daten für einen Tag "incomplete" sind. Dafür prüfe ich heute ob 24 Slots pro Tag vorhanden sind. Das müsste ich dann auch umbauen. Alles nichts wildes, aber etwas was wir bei der Anpassung auf 15min wieder anfassen müssten.

Aber is gut. Der Workaround über die UI scheint gerade der sicherere/pragmatischere Weg zu sein.

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.

Ensure custom tariff datapoints cover every hour
2 participants