Extended Minimal Display Theme to Support Second CP (for DUO) #2568

jotpehenn · 2022-12-29T13:24:05Z

This PR modifies the minimal display theme (which is the same as the default theme for CP only configuration) to support displaying the second CP on DUO devices. Visualization for single CP operation remains unchanged.

Following additional enhancements/fixes apply:

display did not load correctly due to empty variable assignments
there was no way to define the maximum scale value(s) when selecting the minimal display theme
merged maximum scale value selection code for Cards, Gauges and minimal
when device is in "CP only" mode only minimal theme is available (unless parent WB has been selected as display)
added note to CP only display configuration to indicate where to change max scale
fixed small typo

benderl · 2023-01-03T10:22:14Z

Danke für die Umsetzung!
Einziger Kritikpunkt: Es ist nicht auf den ersten Blick ersichtlich, welcher Bogen zu LP1/LP2 gehört.
Kannst Du noch eine Beschriftung ergänzen? Alternativ, wobei ich nicht weiß, ob die verwendete Library das unterstützt, könnten bei zwei Ladepunkten zwei Viertelkreise angezeigt werden. LP1 wäre dann in der linken Hälfte, LP2 rechts zu sehen.
Oder hast Du noch einen anderen Vorschlag dazu?

jotpehenn · 2023-01-07T11:56:52Z

Gefällt mir gut, der Vorschlag mit den Viertelkreisen. Die gauge lib muss dazu aber etwas erweitert werden. Wo ich schon mal dabei war hab ich auch noch die Möglichkeit eingebaut Einheiten an die Label zu heften und die Label innen anzuzeigen. So sieht es jetzt aus:

benderl · 2023-01-07T12:07:53Z

Wow! Sieht echt klasse aus!

web/display/minimal/gaugevalues.php

web/display/minimal/index.php

Kalle01 · 2023-01-10T09:35:14Z

Super Umsetzung!!!
Kannst Du sagen, wann das Verfügbar ist? - Ich gebe den Strom im Augenblick kostenlos ab, weil ich mit den vorhandenen Displays nicht klar komme...

Folgende Ergänzung für halböffentliche Ladepunkte wäre sehr sinnvoll (Ich glaube sogar, dass das Vorschrift ist):
Anzeige des eingestellten Strompreises
Möglicherweise kann man die Anzeige auch optional gestalten:

benderl · 2023-01-10T13:55:03Z

Ich schlage vor, dass der Strompreis in einem separaten PR integriert wid. Hier liegt das Ziel bei der Unterstützung der Duo.

Kalle01 · 2023-01-10T15:42:46Z

Mit einem weiteren PullRequest häbe ich grundsätzlich kein Problem, allerdings geht es hier um das WallboxDisplay "minimal display theme". Und genau hier macht die Angabe des Preises Sinn, da dieses Display das ist, das bei halböffentlichen und öffentlichen Wllboxen zum Einsatz kommt. Bei allen anderen Display Themes ist der Preis eher zu vernachlässigen.
@benderl Entscheide das zusammen mit @jotpehenn.
Mir liegt auch viel daran, dass das Display das Ihr gerade konzipiert schnell verfügbar ist. Notfalls kann ich den Preis auch per Aufkleber anzeigen (wäre aber doof ;-))
Sagt Bescheid, wenn ich einen weiteren PR öffnen soll.

- merge max scale setting for multiple display themes - extend max scale setting for minimal display theme - allow max scale setting even if configured as blind CP

jotpehenn · 2023-01-10T23:02:42Z

@benderl Habe die Anpassungen gemacht. Danke für Deine Geduld...

@Kalle01 sobald dieser PR in die master übernommen wurde, müsste es in der nightly version verfügbar sein. Ich stimme @benderl zu, dass wir diesen PR erst mal durch die Tür bringen sollten bevor wir es mit weiteren Features hinauszögern.

benderl

Die trim() Aufrufe bei Variablen werden jetzt nicht mehr benötigt, da das direkt beim Einlesen der openwb.conf passiert.

Nach dem Merge entferne ich die noch.

Danke für die Umsetzung!

Kalle01 · 2023-01-11T06:31:08Z

@bender und @jotpehenn
alles klar, ich werde einen neune PR machen.
Dies wird für mich der erste ever sein. Bisher hatte ich das Glück, dass ich nur Wünsche äußern musste und andere haben dann die Requests umgesetzt. Geht also nicht so hart mit mir ins Gericht, wenn es nicht optimal formuliert ist oder etwas fehlt ;-)

benderl · 2023-01-11T06:32:08Z

Learning by doing. Alles halb so wild, schließlich haben wir alle mal angefangen.

Kalle01 · 2023-01-12T12:24:26Z

Ich habe gerade ein Update auf nightly gefahren. Es funktioniert noch nicht!
Das Problem scheint im Bereich USB RFID Reader zu liegen. Der schaltet den Ladevorgang bei diesem Theme nicht frei. Im Display äußert sich das darin, dass das Schloss trotz scannen nicht "aufgeht".
Ich habe eine Gegenprobe mit dem Theme Stromfluss gemacht. Hier funktioniert alles.
Ihr könnt mich gerne anrufen. 01795954151 Die Box ist auch in der Cloud.

Außerdem:
Ich habe den PR für die Preisanzeige nicht gebacken bekommen. Ich habe statt dessen einen Issue eröffnet:
#2584

Kalle01 · 2023-01-12T12:45:09Z

Aufffällig weil nicht erwartet:
in der oberen rechten Ecke ist der Button für das Codeschloss aktiv:

Die Freischaltung über Code ist aber ausgeschaltet:

benderl · 2023-01-12T14:02:50Z

Das ist die RFID Eingabemaske.

Kalle01 · 2023-01-12T18:24:57Z

Das hört sich jetzt vielleicht dumm an, aber wofür benötige ich die RFID Eingabemaske bzw. wer braucht denn so was?
Ich habe im Büro einen RFID-Reader mit dem scanne ich die Karten, die ich an die Kunden rausgebe und registriere die Nummern in der Konfiguration der Master-Ladesäule.

Zur Erklärung wie ich getestet habe: Ich habe zwei Duos und habe die jetzt entkoppelt, um auf der ehemaligen Slave das nightly Update zu checken.
Die nightly Säule habe ich dann als Master eingerichtet und mit zwei RFID Tags "bestückt". Dann trat das oben geschilderte Problem auf.
Das Problem, dass das minimal Theme (bei einer Duo) den Ladepunkt nicht entsperrt, ist möglicherweise kein neues Problem. Ich habe dieses Displaytheme bisher nicht benutzt, da die Anzeige für die Duo nicht funktioniert hat. Ich werde das morgen nochmal mit der alten Version testen und dann das Ergebnis hier posten. (Im Augenblick ist es zu dunkel und zu nass)

benderl · 2023-01-12T19:00:27Z

Bitte im Forum diskutieren. Das gehört hier nicht hin. Das Theme hat keinen Einfluss auf den verbauten RFID Leser. Du hast ein anderes Problem.

Kalle01 · 2023-01-30T15:10:32Z

Ich habe den oben geschilderten Fehler gefunden. Defekter Reader :-(.
Aber jetzt habe ich einen kleinen Fehler im Display der dann auch hier hin gehört:

Ich denke der zugewiesene Platz reicht nicht aus, um Zahl plus Einheit darzustellen.
Wenn der Platz für eine korrekte Anzeige nicht ausreichend erweitert werden kann, würde mir auch eine Kommastelle reichen.
Den LP2 konnte ich leider noch nicht testen. Hier fehlt mir in der Zuleitung 1 Phase (in der Verteilung nicht richtig aufgelegt) so dass ich keine zweistelligen kW-Zahlen erreiche.

jotpehenn · 2023-01-31T09:31:57Z

Sollte ein einfacher fix sein. Ich werde mich aber wohl erst am Wochenende drum kümmern können...

benderl · 2023-01-31T19:07:58Z

Ich würde vorschlagen, den Wert in kW mit einer Nachkommastelle anzuzeigen. Genauer wird es eigentlich nicht benötigt. Optional im W, wenn die Leistung kleiner 1kW ist.

Kalle01 · 2023-02-01T08:21:25Z

Der "Elektriker" war da.Bei Ladepunkt 2 sieht es genau so aus wie bei Ladepunkt 2.
Der Effekt scheint an der niedrigen Auflösung des Raspi Display zu liegen. An meinem PC ist das Display o.k.
Evtl. auch einmal für 22 kW checken die "2" braucht mehr Raum wie eine "1"

Kalle01 · 2023-02-01T08:50:43Z

Zur Darstellung: Wie schon geschrieben 1 Nachkommastelle reicht.
Ob man die Einheiten bei kleiner 1KW wechseln muss weiß ich nicht. 0,5 kW ist doch auch verständlich. Kommt das überhaupt vor?

jotpehenn · 2023-02-04T14:33:53Z

#2599

benderl · 2023-02-05T12:10:53Z

Bitte keine weiteren Kommentare, die zu diesem PR keinen Bezug haben. Für Diskussionen und Wünsche gibt es das Forum.

jotpehenn force-pushed the mindisplay branch from c2f6c68 to 99d225e Compare December 29, 2022 13:35

benderl self-assigned this Dec 29, 2022

jotpehenn force-pushed the mindisplay branch from f8b8291 to 5e7b50f Compare January 7, 2023 11:44

benderl requested changes Jan 9, 2023

View reviewed changes

web/display/minimal/gaugevalues.php Outdated Show resolved Hide resolved

web/display/minimal/index.php Outdated Show resolved Hide resolved

jotpehenn force-pushed the mindisplay branch from c1c1d97 to 0e339ff Compare January 10, 2023 20:10

- extended minimal display to support second CP (for DUO)

c017c62

- merge max scale setting for multiple display themes - extend max scale setting for minimal display theme - allow max scale setting even if configured as blind CP

jotpehenn force-pushed the mindisplay branch from 68505d6 to c017c62 Compare January 10, 2023 22:24

Merge branch 'snaptec:master' into mindisplay

5a88494

jotpehenn requested a review from benderl January 10, 2023 22:37

benderl approved these changes Jan 11, 2023

View reviewed changes

benderl merged commit 30df385 into snaptec:master Jan 11, 2023

benderl mentioned this pull request Jan 11, 2023

remove obsolete trim() #2583

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Extended Minimal Display Theme to Support Second CP (for DUO) #2568

Extended Minimal Display Theme to Support Second CP (for DUO) #2568

jotpehenn commented Dec 29, 2022

benderl commented Jan 3, 2023

jotpehenn commented Jan 7, 2023

benderl commented Jan 7, 2023

Kalle01 commented Jan 10, 2023 •

edited

Loading

benderl commented Jan 10, 2023

Kalle01 commented Jan 10, 2023

jotpehenn commented Jan 10, 2023

benderl left a comment

Kalle01 commented Jan 11, 2023

benderl commented Jan 11, 2023

Kalle01 commented Jan 12, 2023

Kalle01 commented Jan 12, 2023

benderl commented Jan 12, 2023

Kalle01 commented Jan 12, 2023

benderl commented Jan 12, 2023

Kalle01 commented Jan 30, 2023

jotpehenn commented Jan 31, 2023

benderl commented Jan 31, 2023

Kalle01 commented Feb 1, 2023 •

edited

Loading

Kalle01 commented Feb 1, 2023

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

jotpehenn commented Feb 4, 2023

This comment was marked as off-topic.

benderl commented Feb 5, 2023

Extended Minimal Display Theme to Support Second CP (for DUO) #2568

Extended Minimal Display Theme to Support Second CP (for DUO) #2568

Conversation

jotpehenn commented Dec 29, 2022

benderl commented Jan 3, 2023

jotpehenn commented Jan 7, 2023

benderl commented Jan 7, 2023

Kalle01 commented Jan 10, 2023 • edited Loading

benderl commented Jan 10, 2023

Kalle01 commented Jan 10, 2023

jotpehenn commented Jan 10, 2023

benderl left a comment

Choose a reason for hiding this comment

Kalle01 commented Jan 11, 2023

benderl commented Jan 11, 2023

Kalle01 commented Jan 12, 2023

Kalle01 commented Jan 12, 2023

benderl commented Jan 12, 2023

Kalle01 commented Jan 12, 2023

benderl commented Jan 12, 2023

Kalle01 commented Jan 30, 2023

jotpehenn commented Jan 31, 2023

benderl commented Jan 31, 2023

Kalle01 commented Feb 1, 2023 • edited Loading

Kalle01 commented Feb 1, 2023

This comment was marked as off-topic.

This comment was marked as off-topic.

This comment was marked as off-topic.

jotpehenn commented Feb 4, 2023

This comment was marked as off-topic.

benderl commented Feb 5, 2023

Kalle01 commented Jan 10, 2023 •

edited

Loading

Kalle01 commented Feb 1, 2023 •

edited

Loading