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

Planner: remove special handling of last slot #6970

Merged
merged 7 commits into from
Apr 2, 2023
Merged

Conversation

andig
Copy link
Member

@andig andig commented Mar 20, 2023

Fix #6960

  • fix tests
  • fuse partial slots with previous or following slot
  • make fixed tariff return hour slots

@andig andig added the enhancement New feature or request label Mar 20, 2023
@andig andig changed the title Remove special handling of last slot Planner: remove special handling of last slot Mar 20, 2023
@andig
Copy link
Member Author

andig commented Mar 20, 2023

So ganz glücklich bin ich trotzdem nicht. Zur Einordnung: der “last slot” ist der teuerste aus Kostensicht. Es kann zeitlich aber durchaus der erste oder eben ein beliebiger sein!
Mir ist aufgefallen, dass die Änderung Lücken nicht verhindert, sondern eben nur an eine andere Stelle schiebt (wenn man nur 30min eines 1h Slots braucht landen die jetzt halt am Anfang und nicht am Ende). prinzipiell gibt das nochmal mehr Zeit für Erkenntnisgewinn und auch in der zweiten halben Stunde könnte ja noch geladen werden.
Wenn wir allerdings nur 5min des Slots nutzen sollten (was wir aktuell im Loadpoint unterdrücken falls das zu Ladeunterbrechungen führen würde) sieht das Bild anders aus. Die an den Anfang zu legen obwohl sich vllt. am Ende ein weiterer Slot anschließt wäre kontraproduktiv.
Vielleicht sollten wir das einfach gar nicht so machen, sondern nicht voll genutzte Slots jeweils mit den zeitlich davor oder danach kommenden- falls diese zusammen hängen- verschmelzen.
In Summe- es bleibt schwierig. Vielleicht lassen wir es auch einfach wie es ist. Feedback welcome.

/cc @Hofyyy @schenlap

@andig andig marked this pull request as draft March 20, 2023 21:20
@schenlap
Copy link
Contributor

Ja, schwierig :-)

Alternative wäre einen ganzen zusammenhängenden Block zu suchen und in diesem zu laden. Wäre in Summe viel einfacher umzusetzen weil es keine Ausnahmen gibt. Man nimmt einen Block mit der berechneten Ladezeit und beginnt diesen von spät nach früher zu verschieben (z.B 15 min Raster) und schaut wo es in Summe am günstigsten ist. Das kann dann teurer sein als jetzt, dafür ist es immer zusammenhängend. Es kann aber auch günstiger sein weil der Tesla nicht wieder neu heizen muss. Ich denke aber dass es die Diskusionen beenden würde und auch eine gute und verständliche Lösung wäre.

@andig
Copy link
Member Author

andig commented Mar 21, 2023

Auch gute Strategie solange es keine mehrtägigen Pläne gibt? Könnten wir mal als Prototyp anbieten.

@Hofyyy
Copy link
Contributor

Hofyyy commented Mar 21, 2023

Ich sehe den weiteren vorteil, das es der mitbewerb genau so macht und wir weniger rückfragen / support auskommen haben

@andig
Copy link
Member Author

andig commented Mar 21, 2023

Das stimmt. Ich hab das jetzt auch in #5639 (comment) nochmal hoch gezogen.

@andig
Copy link
Member Author

andig commented Apr 1, 2023

@schenlap @Hofyyy könnt ihr mal bitte einen Blick auf 8e408ea werfen?

Die "late slot" Logik ist jetzt dynamisch insofern als das sie mal schaut, was in der zeitlichen Umgebung des preislich teuersten (und damit letzten im Plan) Slots passiert. Falls es einen genau davor gibt oder keinen genau danach dann starten wir früh (bisher: immer spät). Nur wenn der Slot völlig isoliert steht starten wir spät.

Außerdem werden "kurze" Slots nur noch ignoriert wenn sie allein stehen (wobei wir dann allerdings nicht auf die Gesamtlänge schauen...)

Was meint ihr?

@andig andig marked this pull request as ready for review April 1, 2023 15:20
@andig andig force-pushed the feature/plan-slot-end branch from cdfa8ca to f81d8cc Compare April 2, 2023 15:55
@andig andig merged commit cdc7cf0 into master Apr 2, 2023
@andig andig deleted the feature/plan-slot-end branch April 2, 2023 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Planner: start last slot early instead of late
3 participants