-
-
Notifications
You must be signed in to change notification settings - Fork 640
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
Sessions UI: visualization, month/year charts #15902
base: master
Are you sure you want to change the base?
Conversation
Das sieht richtig genial aus. Bei kurzem Test ist es mir allerdings nicht gelungen, in diesem Branch die "Charging Sessions" zu öffnen- Menüclick tut gar nichts mehr. Console:
|
Vorsichtshalber die Frage: |
@VolkerK62 was hat das mit Grafiken zu tun? Nix... |
ok, ok. bitte nicht schlagen 😄 |
@VolkerK62 klar, CSV gleibt weiterhin da |
|
Geht weder Safari noch Chrome:
|
Wunderschön ;-) |
Merkwürdig. Ich schau mal ob ich das irgendwie reproduziert bekommen. |
Bin begeistert, super gelungen. 👍🤗 ist echt Hammer geworden |
@StevieC121176 das ist ja noch nicht fertig 😄. Da geht noch was. |
Ich würde sagen, da bleiben keine Wünsche offen. |
Sieht super aus 👍 |
Bzgl. Einheiten: könnten wir bei kWh bleiben, zumindest solange bis die Zahlen 5-stellig werden? Das ist die leichter zu greifende Einheit? |
…und sollen wir von Kosten statt Preis sprechen? |
Lass uns damit mal an den Start gehen. Um Gegensatz zum Balkendiagramm hat das den Vorteil, dass es sehr gut mit unterschiedlichen vielen Einträgen klarkommt. Das sieht mit 2 Einträgen gut aus, aber auch noch mit 8. Ein Balkendiagramm mit zwei Einträgen wirkt schnell komsich. |
Ja hier war die Legende mit den Prozenten auch noch nicht optimal. Schau dir mal den Screenshot zwei weiter oben an. Mit der aktualisierten Legende (Prozente + Energie) passt die Überschrift auch besser. |
Die Einheiten wären alle 2.000kWh. Was spricht hier gegen kWh? |
Konsistenz (und Platz). Wenn wir das ändern sollten wir das global ändern und nicht nur hier. |
Links steht noch Preis |
energySubTitle = "{energy} total" | ||
energyTitle = "{percent} Solar Energy" | ||
groupedCo2TitleByGroup = "CO₂-Amount {byGroup}" | ||
groupedPriceTitleByGroup = "Total Cost {byGroup}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cost im Key
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope. Technisch ist bei uns "Cost" der Oberbegriff für "Co2" und "Preis" (siehe auch Smart Cost Limit). Wie wir das dann konkrete in der UI betiteln ist ein anderes Thema.
Ich verstehe nicht genau was du damit meinst. Den Ladepreis (ct/kWh) würde ich so lassen. Im Sprachgebrauch ist "Preis pro Kilowattstunde" deutlich gebräuchlicher als "Kosten pro Kilowattstunde". |
Ich kenne Fahrtkosten, keine Preise? Auch Spritkosten. Auf jeden Fall ist es in #15902 (comment) jetzt gemischt. Entweder oder? |
s/nach/je/ ? Beides möglich? Soll die Autoformatierung bei 4 Stellen auf das Nachkomma verzichten? |
da bin ich leidenschaftslos. Ist geändert |
Ich sehen deinen Punkt. Technisch machen wir damit aber ein größeres Fass auf. Dann müssen wir irgenwie sicherstellen, dass bei tabellarischen Ansichten (Übersicht Ladevorgänge, Energieflussdiagramm, und hier) nicht nur die Einheiten synchron sind sondern sich auch die Anzahl der Nachkommastellen immer synchron sind. Da seh ich gerade keinen Weg der nicht messy ist. Lass uns das Thema hier rausziehen. Wenns kurz und übersichtlich sein soll können wir sonst wieder auf MWh zurück. |
Wir reden hier doch über zwei unterschiedliche Dinge: Der errechente "Preis pro kWh" (Vergleichbar mit Preisen an Ladesäule oder Stromtarif / Arbeitspreis / Strompreis) und deine angefallenen "Kosten" für einen bestimmten Verbrauch. |
wip
TODOs
mobile.stats.webm