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

Refactor session settings and apis (BC) #10335

Merged
merged 108 commits into from
Dec 2, 2023
Merged

Conversation

andig
Copy link
Member

@andig andig commented Oct 15, 2023

Fix #10330, fix #9235, fix #9245, fix #9641, fix #10696, fix #10650

Functional changes:

  • introduce dedicated plan soc/energy (used by planner)
  • UI: dedicated plan indicator in progress bar
  • renamed target soc/energy to limit soc/energy (hard upper limit)
  • new behavior: solar charging will continue after plan is fulfilled if limit is higher
  • support for mixed setups (vehicle with soc but without capacity, lp soc + guest vehicle)
  • prepared UI for multiple plans (coming later)

TODO

  • remove limitSoc from ActionConfig and add to vehicle API
  • remove site. prefix from settings (BC break)
  • add limitSoc to Arrival UI @naltatis
  • adapt ui to api structure @naltatis
  • adapt and extend tests @naltatis
  • add plan error messages @naltatis
  • new plan ui (api change, prepare for multiple plans) @naltatis
  • ui for energy based plans @naltatis
  • discuss removal of resetondisconnect @naltatis
  • update effective values when needed
  • ensure vehicle can only be attached to one loadpoint
  • vehicle range gets lost when changing vehicles
  • publish vehicleName at loadpoint
  • fix config endpoint (newDeviceHandler) @andig

BC Breaks

Global:

  • uri removed, use network instead

Site:

  • UI configuration needs be redone since configuration keys have changed (removed the site. prefix from db)
  • replaced pvs by pv
  • replaced batteries by battery
  • removed vehicles- no longer needed
  • removed prioritySoc (adjustable in ui)
  • removed bufferSoc (adjustable in ui)
  • removed bufferStartSoc (adjustable in ui)

Loadpoint:

  • mode is now persisted
  • resetondisconnect removed- if mode is configured it is used when vehicle is disconnected
  • removed soc.min (adjustable in plan ui -> Arrival)
  • removed soc.target (adjustable in by plan ui -> Arrival)

@andig andig added the infrastructure Basic functionality label Oct 15, 2023
@andig andig changed the title Refactor session settings and apis Refactor session settings and apis (BC) Oct 15, 2023
@naltatis naltatis self-requested a review October 15, 2023 16:06
@andig andig force-pushed the chore/refactor-settings-and-apis branch from 2cb07ef to 9aca861 Compare October 21, 2023 12:05
@andig andig marked this pull request as draft October 22, 2023 15:10
@andig andig force-pushed the chore/refactor-settings-and-apis branch 2 times, most recently from 93b456e to 3c994c5 Compare October 23, 2023 09:22
@andig andig force-pushed the chore/refactor-settings-and-apis branch 5 times, most recently from 22d000c to 5b2fbbe Compare November 1, 2023 21:55
server/http.go Outdated Show resolved Hide resolved
core/loadpoint_api.go Outdated Show resolved Hide resolved
@andig andig force-pushed the chore/refactor-settings-and-apis branch from 490d0c7 to 88a42b0 Compare November 6, 2023 19:08
@andig
Copy link
Member Author

andig commented Nov 9, 2023

@naltatis was ist hier noch offen?

@naltatis
Copy link
Member

@andig kannst du vehicleName am LP noch mit publishen. Dann muss ich den Lookup nicht über vehicleTitle machen.

@naltatis
Copy link
Member

@andig Beim Fahrzeugwechsel scheint es noch Probleme zu geben. Der Fahrzeugwechsel über die UI funktioniert. Allerdings kann aktuell ein Fahrzeug an mehreren LPs gleichzeitig angemeldet sein. Vorher war das anders. Da wurde das Fahrzeug vom Loadpoint entfernt, falls es schon woanders verbunden war.

Beim Publishing der Reichweite (vehicleRange) scheint es auch ein Problem zu geben. Initial ist alles korrekt. Wechselt man über die UI das Fahrzeug so verschwindet die Reichweite obwohl das ausgewählte Fahrzeug eine Reichweite hat.

@naltatis
Copy link
Member

naltatis commented Nov 11, 2023

Bildschirmfoto 2023-11-11 um 20 29 00

Inkl. Reichweitenschätzung, sofern möglich. Siehe #10650

@andig
Copy link
Member Author

andig commented Nov 12, 2023

@andig kannst du vehicleName am LP noch mit publishen. Dann muss ich den Lookup nicht über vehicleTitle machen.

Als vehicleId aufgenommen. Dass das zufällig der name aus der Config ist würde ich gerne verschweigen?

@naltatis
Copy link
Member

Dass das zufällig der name aus der Config ist würde ich gerne verschweigen?

Verstehe ich nicht. Wieso hier nicht name? id nutzen wir in der Config UI "nur" für die optionale DB-ID, name als universellen Identifiert (yaml & db). Hier wollen wir je letzteres.

@andig
Copy link
Member Author

andig commented Nov 12, 2023

Kannst du mir sagen wo wir das schon als name bezeichnet haben? Ich steh grad auf dem Schlauch.

@naltatis
Copy link
Member

@andig all clear

core/coordinator/adapter.go Outdated Show resolved Hide resolved
core/keys/loadpoint.go Outdated Show resolved Hide resolved
core/keys/loadpoint.go Outdated Show resolved Hide resolved
core/keys/loadpoint.go Outdated Show resolved Hide resolved
@andig
Copy link
Member Author

andig commented Dec 2, 2023

@naltatis @GrimmiMeloni lohnt es für

removed the site. prefix from db

noch eine Migration zu schreiben, die genau 1x läuft?

@GrimmiMeloni
Copy link
Collaborator

@naltatis @GrimmiMeloni lohnt es für

removed the site. prefix from db

noch eine Migration zu schreiben, die genau 1x läuft?

Was ist denn für den normal Anwender die Auswirkung wenn wir diese Migration nicht machen?

@andig
Copy link
Member Author

andig commented Dec 2, 2023

I.W. Batteriesettings sind neu einzugeben. Kein grosses Ding.

@GrimmiMeloni
Copy link
Collaborator

I.W. Batteriesettings sind neu einzugeben. Kein grosses Ding.

Dann würde ich das nicht migrieren.

Copy link
Member

@naltatis naltatis left a comment

Choose a reason for hiding this comment

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

Krasser Change. Irgendwas haben wir bestimmt übersehen. Aber durch Draufgucken find ich gerade nichts mehr. Lets try.

@andig andig merged commit d63dec9 into master Dec 2, 2023
7 checks passed
@andig andig deleted the chore/refactor-settings-and-apis branch December 2, 2023 15:56
@andig
Copy link
Member Author

andig commented Dec 2, 2023

So be it :) Morgen werden wir in einer anderen Welt aufwachen :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment