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

UI : kleiner Bug : Hausbatterie : Netzladen : Preisgrenze #17975

Open
1 task done
vkr2791 opened this issue Dec 31, 2024 · 7 comments
Open
1 task done

UI : kleiner Bug : Hausbatterie : Netzladen : Preisgrenze #17975

vkr2791 opened this issue Dec 31, 2024 · 7 comments
Assignees
Labels
ux User experience/ interface

Comments

@vkr2791
Copy link

vkr2791 commented Dec 31, 2024

Describe the bug

Bei einem Strompreis von z.B. 20 ct greift nicht die Einstellung "<= 20ct"
hier fehlt wohl das "<=" im Code, siehe Screenshots:

grafik

Mit "<=20,5 ct" greift es dann:

grafik

Steps to reproduce

...

Configuration details

-

Log details

-

What type of operating system or environment does evcc run on?

Docker container

Nightly build

  • I have verified that the issue is reproducible with the latest nightly build

Version

0.132.0

@VolkerK62
Copy link
Contributor

Da wird der Preis vermutlich 20,04 ct betragen

@andig andig added the ux User experience/ interface label Dec 31, 2024
@andig
Copy link
Member

andig commented Dec 31, 2024

Sollten wir im Code auf 1 Nachkommastelle runden? Oder anders visualisieren?

@StefanSchoof
Copy link
Contributor

Ist runden hier überhaupt sinnvoll? Warum nicht für eine Stunde genau den Preis den der Anbieter haben will angezeigen?

@naltatis
Copy link
Member

Ist runden hier überhaupt sinnvoll? Warum nicht für eine Stunde genau den Preis den der Anbieter haben will angezeigen?

Weil wir sonst je nach Schnittstelle viele Nachkommastellen anzeigen.

@naltatis
Copy link
Member

Sollten wir im Code auf 1 Nachkommastelle runden? Oder anders visualisieren?

Die Visualisierung und die Logik/Regelung sind in sync. Heißt er würde auch in dieser Stunde nicht laden. Wir könnten hier aber in der Tat direkt beim einlesen der Daten schon Runden/Abschneiden. Gerade wenn wir es nicht genauer im UI anzeigen wollen führt das dann zu weniger Verwirrung.

Ist allerdings nicht total trivial, weil die Präzision in der wir das Anzeigen abhängig ist von der Währung (ct wenn EUR). Die gesamte Liste gibts hier: https://github.com/evcc-io/evcc/blob/master/assets/js/mixins/formatter.js#L2-L15

@StefanSchoof
Copy link
Contributor

Weil wir sonst je nach Schnittstelle viele Nachkommastellen anzeigen.

Ja, aber über verschiedene Währungen ist es unmöglich die Anzahl der für den Nutzer signifikanten Nachkommastellen zu bestimmen. Z.B. hat der Kuwait Dinar hat eine ganz andere Auflösung als der Russische Rubel.

@vkr2791
Copy link
Author

vkr2791 commented Dec 31, 2024

Ach so, das Problem besteht auch bei der Preisgrenze für billige Netzenergie im Modus PV:

grafik

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ux User experience/ interface
Projects
None yet
Development

No branches or pull requests

5 participants