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

Grid meter optional: improved no-solar use case, UI configuration #14341

Merged
merged 4 commits into from
Jun 22, 2024

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Jun 13, 2024

Config UI: allow statup without grid meter in evcc.yaml > db configuration possible
Dynamic Pricing: optimize UI for use cases without solar

fixes #9871

  • 🆙 allow server start without grid (or pv) meter being configured
  • 📱 make grid meter ui-configurable when no yaml entry exists
  • 💡 rename pv-mode to smart mode if only a dynamic tariff exists
  • 🔅 remove production, feed-in, ... values if pv is not possible (no grid, no pv)

smart cost only

no grid, no pv

Todos

  • PV > Smart, when no grid and pv
  • remove Min+PV, when no grid and pv
  • add e2e tests

@naltatis naltatis added enhancement New feature or request ux User experience/ interface prio Priority labels Jun 13, 2024
@naltatis naltatis requested a review from andig June 13, 2024 15:35
@naltatis naltatis mentioned this pull request Jun 13, 2024
22 tasks
@StefanSchoof
Copy link
Contributor

Erlaubt dies auch ein Starten, wenn eins der Meter gerade Fehler hat, wie neue IP die man anpassen muss?

@naltatis
Copy link
Member Author

Nein, das Startverhalten bei fehlerhaften Metern und Chargern ist noch mal ein anderes Thema.

@andig
Copy link
Member

andig commented Jun 15, 2024

Ok. Sollten wir in dem Fall nicht die Site nicht starten sondern das wie auch andere Fehler behandeln? Mir ist nicht klar, warum wir die site ändern müssen?

@andig andig marked this pull request as draft June 16, 2024 15:24
@naltatis
Copy link
Member Author

Sollten wir in dem Fall nicht die Site nicht starten sondern das wie auch andere Fehler behandeln?

Ich verstehe nicht genau, was du damit meinst.

Hintergrund: Aktuell kann man das Grid-Meter über Config UI nicht setzen/ändern, weil wir gar nicht sauber hochfahren, ohne dass es ein Grid (oder PV) Meter gibt. Ist die Site nicht initialisiert können wir auch kein Grid Meter per API setzen/ändern.

Für die Nutzung von evcc nur mit dynamischen Strompreisen (siehe #9871) sind die Meter technisch auch nicht erforderlich.

@andig
Copy link
Member

andig commented Jun 19, 2024

Bleiben also zwei Optionen:

  • wir verschieden alle Site/Loadpoint config APIs nach /config (dann muss site nicht starten)
  • oder wir erlauben die Initialisierung, starten aber nicht die Run() Methode

?

@naltatis
Copy link
Member Author

@andig vielleicht übersehen ich da was. Aber was spricht denn dagegen, die Site zu starten ohne Grid o. PV Werte zu haben? Für die Verwendung von reiner Steuerung nach dynamischen Strompreisen ist das ja nicht erforderlich.

@andig
Copy link
Member

andig commented Jun 19, 2024

Ich kann mal mindestens nicht sagen ob das geht. Was ist mit circuits wenn die Zähler nicht funktionieren oder gar nicht da sind?

@andig
Copy link
Member

andig commented Jun 19, 2024

Gleiche Diskussion wie #14474. Restart erhöht die Chance auf funktionierendes System, hat aber auch Nebenwirkungen.

@naltatis
Copy link
Member Author

@andig Dieser PR hat nichts mit nicht erreichbaren Devices zu tun. Hier geht um Config UI und den Zustand, dass man evcc startet ohne in der yaml einen Zähler zu haben: Ersteinrichtung.

@andig
Copy link
Member

andig commented Jun 20, 2024

Ok. Dann könnten wir Steuerung nach dynamischen Strompreisen aber hier erstmal out of scope nehmen und auch auf start der Site verzichten? Sie müsste nur existieren damit die Apis gibt? Oder eben diese verschieben.

@andig andig added the needs documentation Triggers issue creation in evcc-io/docs label Jun 20, 2024
@naltatis naltatis changed the title Config UI: allow start without grid or pv meter Grid meter optional: improved no-solar use case, UI configuration Jun 21, 2024
@naltatis naltatis marked this pull request as ready for review June 21, 2024 21:06
@naltatis
Copy link
Member Author

Wie besprochen ist hier jetzt auch die Anpassung für Dynamische Preise ohne Grid/PV drin.
Der PV-Modus ist hier dann in Smart umbenannt.
Ist weder Grid/PV/DynamischerTarif vorhanden gibte es nur Aus/Schnell

@andig andig merged commit 280976c into master Jun 22, 2024
6 checks passed
@andig andig deleted the feature/start_without_grid_pv branch June 22, 2024 08:21
@sasquaatch
Copy link

ich habe bei "mir" einen dynamsichen Tibber Tarif und deswegen auch einen Grid Meter, wenn der eingetragen ist fehlt bei mir "smart" und stattdessen habe ich "PV" und "Min+PV", allerdings habe ich gar keine PV Anlage.
Die Frage wäre natürlich für was der Grid Meter eingetragen wird wenn man gar kein PV hat, aber wenn ich mir den Verbrauch in meiner Wohnung anschaue, dann öffne ich lieber EVCC als die Tibber App. Es wäre also wünschswert würde es auch eine Variante mit Grid und ohne PV geben um "smart" als Auswahl haben zu können

@naltatis
Copy link
Member Author

Ja, das wird auf jeden Fall kommen. Die Lösung mit "smart" wenn es kein Grid gibt ist ein erster Schritt. Im Zuge von #3530 werden wir das noch größer angehen.

Hintergrund: Sobald es ein Grid Meter gibt, hat evcc alles, um PV-Steuerung zu machen. Heißt, wenn du jetzt Strom in deinem Haus erzeugen würdest (Balkon PV in die Steckdose). Würde das alles schon klappen, ohne dass du eine PV konfigurierst. Daher gehen wir, aktuell, bei einem vorhandenen Grid-Meter immer davon aus, dass es auch Einspeisung geben kann.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs documentation Triggers issue creation in evcc-io/docs prio Priority ux User experience/ interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improvements for dynamic tariff charging without PV installation
4 participants