Alles klar, ich nehme das, was ich gemacht habe, wieder auf. Das sollte schnell gehen.
@pierre-gilles
In solchen Fällen, muss ich dann einen neuen PR starten? (Ich glaube, da gibt es keine große Wahl, da ich nicht von seinem Branch arbeiten kann?)
Ich muss nur noch das Tooltip finalisieren => schwer, Infos zu solchen Daten zu finden!! Aber ich denke, ich habe die richtige Quelle gefunden ^^
https://codepen.io/andrussuitsu/pen/qBKEqzr?editors=1111
Wäre das so in Ordnung?
Ja, natürlich einen neuen PR!
PR: Add binary chart by Terdious · Pull Request #2094 · GladysAssistant/Gladys · GitHub
Aber die Namen sind wirklich winzig:
An sich ist die PR für die zoombare Anzeige im Dashboard relativ einfach, ohne die gesamte Historie und die Datumsangaben zu implementieren! Vielleicht lohnt es sich, eine kleine PR dazu zu machen @pierre-gilles? Und dann können wir das Thema „Verbesserung der Kurven“ in einem zweiten Schritt anhängen?
Gibt es keine andere Möglichkeit, die Titel anzuzeigen?
Ja 100%, ich bin für kleine PRs, das ist das Beste, um zu debuggen, und damit die Funktionen so schnell wie möglich beim Endnutzer ankommen.
Kann der Name direkt unter der Grafik-Linie sein?
Ich suche alles, was man in der Dokumentation machen kann ^^ Nicht einfach, aber wir werden es finden ^^
@pierre-gilles, wäre das hier passend:
yaxis.labels.style.cssClass (Options (Reference) > yaxis | ApexCharts.js) muss man direkt am SVG herumfummeln. Ich habe also den „device“-Teil vom „feature“-Teil getrennt, um einen Zeilenumbruch zu erzeugen.
Ich wollte die Schriftgröße im Diagramm unter 2 Binärstellen reduzieren. Was denkst du? Auf dem Bild sind die Namen der rechten Grafik doch total zusammengedrückt, oder?
Ehrlich gesagt ist das schon mal gar nicht so schlecht 
Das ist ja richtig gut, finde ich! Tolle Arbeit! Ist das alles im PR?
Ich finde, der Zeilenumbruch sieht gut aus, ich bin persönlich nicht schockiert vom rechten Diagramm.
Auf jeden Fall ist es unmöglich, einen Gerätenamen + eine Funktion in allen Fällen vollständig unterzubringen, meiner Meinung nach!
Noch nicht ^^ ich warte auf die Freigabe ^^ ich versuche es heute Abend!!
Aber der Schreibstil stört mich, oder?
EDIT: Es ist push. @pierre-gilles, denkst du, du könntest das schnell für mich überprüfen?
Ich habe dir gerade eine Review gemacht, momentan nur funktional, leider nicht sehr aussagekräftig ^^
Die UI friert bei mir komplett ein, ich kann die Funktion nicht zum Laufen bringen 
Details zu meinem Test:
Vielen Dank, ich führe die Tests mit wenig Wert fort.
Ich kann es reproduzieren … Aber was ich nicht verstehe, ist, dass das nichts direkt mit dem PR zu tun hat. Es ist die Anfrage, laut dem console.log, den ich direkt danach eingefügt habe, die sehr lange dauert, nur wenn nicht genug Werte vorhanden sind (< 3 oder 5 Werte) … trotzdem haben wir auf dieser Seite nichts verändert ..
Ich werde auf der Serverseite nachschauen, ob es einen besonderen Grund gibt.
EDIT:
Leer:
Hallo @pierre-gilles,
Es handelte sich also um einen grafischen Bug, die Erstellung des Diagramms läuft vermutlich ins Leere, weil es das rangeBar-Diagramm nicht erstellen kann, wenn die Daten unter dem Min/Max-Bereich der Grundeinstellung (die 5 Minuten betragen sollten) liegen. Sobald es mehr als 3/4 Minuten Daten hatte, war es nicht in der Lage, die Daten im richtigen Format darzustellen und zeigte keine Daten an, obwohl die Kurve erschien. Und das bis zu 10 Minuten Daten.
Laut allen meinen Tests (die Zeit, die ich damit verbracht habe 
^^) sollte die Hinzufügung in den Optionen von:
min: Math.min(...series.flatMap(s => s.data.map(d => d.y[0]))),
max: Math.max(...series.flatMap(s => s.data.map(d => d.y[1]))),
Funktionieren. Ich lasse es den ganzen Tag laufen, um sicherzugehen.
EDIT: Die Änderungen wurden gepusht
@Terdious Danke für die Untersuchung und die Änderungen!
Ich habe gerade gepullt, habe deinen PR getestet, aber es wird nichts angezeigt, obwohl die API Werte zurückgibt:
Hallo @pierre-gilles,
Entschuldigung, ich hatte den Minimalwert für die Anzeige der Kurve (>1) nach meinen Tests nicht wieder geändert. Das ist jetzt erledigt. Entschuldige die Umstände.
Soll ich eine spezifische Nachricht für die Binärdatei hinzufügen, um darauf hinzuweisen, dass mindestens 2 Werte benötigt werden (sonst wird eine leere Kurve angezeigt)?
Ich würde gerne eine ‹ heightAdditional › für die Anzeige der Binärdateien hinzufügen, wenn mehr als 3 ausgewählt sind. So in etwa:
Denn sonst sieht man so etwas:
Was hältst du davon?
(Bitte beachte das Icon oben rechts nicht, das ist mir während der Tests im PR für die erweiterte Kurvenansicht aufgefallen.)
Ach ja, tatsächlich sollte man die Größe dieses Widgets nicht begrenzen. Wie hast du das geändert?
Warum würde eine einzelne Wert eine leere Kurve anzeigen?
Wenn dein Sensor die Tür gerade geöffnet hat, sollte die Tür dann nicht als „geöffnet“ markiert werden?
Das ist ein rangeBar, also müssen zwingend 2 Zeitpunkte angegeben werden. Das Diagramm zeigt ‹ an › von Datum zu Datum oder ‹ aus › von Datum zu Datum. Das war die Grundentwicklung von @Lokkye und mir scheint, dass dies auch das erwartete Verhalten ist. Oder?
EDIT: Lol, ich habe mich in deinen Antworten verheddert ^^ Ich push die Änderung und sage dir Bescheid.
