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

v3: HistoryGraph #783

Closed
mcuiobroker opened this issue Jun 12, 2021 · 30 comments
Closed

v3: HistoryGraph #783

mcuiobroker opened this issue Jun 12, 2021 · 30 comments
Assignees
Labels
bug 🐛 Something isn't working Module HistoryGraph Module HistoryGraph

Comments

@mcuiobroker
Copy link
Collaborator

mcuiobroker commented Jun 12, 2021

Es wird nichts angezeigt. Da muss ich nochmal genauer schauen. (TEST-System)
UPDATE huetemperaturesensor6_00e6c temperature
(anonymous) @ vendor.d87ce6a3.js:129

Failed to decode downloaded font: http://192.168.178.70:8082/jarvis/fonts/flUhRq6tzZclQEJ-Vdg-IuiaDsNcIhQ8tQ.044abb9e.woff2 index.html#/corona9:1 OTS parsing error: Size of decompressed WOFF 2.0 is less than compressed size index.html#/corona9:1 Failed to decode downloaded font: http://192.168.178.70:8082/jarvis/fonts/flUhRq6tzZclQEJ-Vdg-IuiaDsNcIhQ8tQ.044abb9e.woff2 index.html#/corona9:1 OTS parsing error: Size of decompressed WOFF 2.0 is less than compressed size index.html#/corona9:1 Failed to decode downloaded font: http://192.168.178.70:8082/jarvis/fonts/materialdesignicons-webfont.e9db4005.woff2 index.html#/corona9:1 OTS parsing error: Size of decompressed WOFF 2.0 is less than compressed size index.html#/corona9:1 Failed to decode downloaded font: http://192.168.178.70:8082/jarvis/fonts/materialdesignicons-webfont.e9db4005.woff2 index.html#/corona9:1 OTS parsing error: Size of decompressed WOFF 2.0 is less than compressed size vendor.d87ce6a3.js:129 UPDATE huetemperaturesensor6_00e6c temperature

@mcuiobroker mcuiobroker added the bug 🐛 Something isn't working label Jun 12, 2021
@Zefau
Copy link
Owner

Zefau commented Jun 12, 2021

Welchen Adapter nutzt du? History, Influx oder sql? Hast du den angegeben?

@Zefau Zefau added this to the ROADMAP v3.0.0 milestone Jun 12, 2021
@mcuiobroker
Copy link
Collaborator Author

image

@Zefau
Copy link
Owner

Zefau commented Jun 13, 2021

Fixed with v3.0.0-alpha.7

INFO: Die ganzen Optionen sind noch ohne Wirkung, aber der Chart sollte angezeigt werden.

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Jun 13, 2021

Wird angezeigt: Schrift im Dark Modus nicht zu lesen:
image
Kann da irgendwo ein X zum schließen hin?
image
Höhe variabel? Und zweiter Wert wird in der Legende nicht angezeigt?
image

@mcuiobroker
Copy link
Collaborator Author

Bei der Konvertierung zu v3 werden die Farben nicht vordefiniert, so dass alle Linien in der gleichen Farbe erscheinen:
image

@mcuiobroker
Copy link
Collaborator Author

v3.0.0-alpha.9 Beim Tabwechsel verschwindet der Graph:
jarvis3_HistoryGraph_nach_tabWecshel_weg

@Zefau
Copy link
Owner

Zefau commented Jun 26, 2021

sollte mit v3.0.0-alpha.20 nicht mehr verschwinden

@mcuiobroker
Copy link
Collaborator Author

Stimmt, verschwindet nicht mehr. Problem mit den DP bleibt.
Weiterhin haben die unterschiedlichen DP die gleiche Farbe und es wird nur einer bei mir angezeigt in der Legende oben, obwohl 2 da sind.

@Zefau Zefau added the #prio:HIGH ❗ Priority: HIGH :exclamation: label Jul 14, 2021
@Zefau
Copy link
Owner

Zefau commented Jul 15, 2021

Wird angezeigt: Schrift im Dark Modus nicht zu lesen:

Sollte bereits mit der aktuellen alpha passen, kannst du das bestätigen?

Höhe variabel? Und zweiter Wert wird in der Legende nicht angezeigt?

Das passiert bei gleichen Label-Bezeichnern in eCharts.

Bei der Konvertierung zu v3 werden die Farben nicht vordefiniert, so dass alle Linien in der gleichen Farbe erscheinen:

Sollte ebenfalls behoben sein. Kannst du das bestätigen?

@mcuiobroker
Copy link
Collaborator Author

Ja man muss entsprechend die Label anpassen, dann passt es auch mit den Farben. Mit der Konvertierung prüf ich ebenfalls morgen.

@mcuiobroker
Copy link
Collaborator Author

Also es kommt wieder zu dem Problem des Verschwindens der kompletten Historygraph-Anzeige, nur Überschrift bleibt:
image
Bei der Konvertierung werden die Label nicht mit den Wertenamen gefüllt:
DP mit History: coronavirus-statistics.0.Germany.Stadt.xxx.cases7_per_100k
DP2 : coronavirus-statistics.0.Germany.Stadt.xxx.deaths
er nimmt davon jeweils "corona" anstatt "cases7_per_100k" bzw "deaths".

@mcuiobroker
Copy link
Collaborator Author

Mit v3.0.0-alpha.29 wird ein Fehler angezeigt:

image

@Zefau
Copy link
Owner

Zefau commented Jul 17, 2021

aktuell lädt HistoryGraph alle historischen Daten eines Datenpunkts ohne zeitliche Einschränkung. Das scheint einfach zu viel zu sein, da muss ich nochmal ran und eine Eingrenzung wie in v2 hinzufügen.

@Zefau Zefau changed the title v3: HistoryGraph keinerlei Anzeige v3: HistoryGraph Jul 18, 2021
@Zefau Zefau added the Module HistoryGraph Module HistoryGraph label Jul 18, 2021
@Zefau
Copy link
Owner

Zefau commented Jul 31, 2021

Mit v3.0.0-alpha.41 ist das Modul jetzt v3 ready (sofern bug-free).

Nun ist wie in v2 eine Zeitangabe festzulegen. Vorher wurden alle History-Daten eines Datenpunkts geladen, was einfach zu viele Daten sind und zu in-performant ist. Nun kann ein festes Von-Bis-Datum ausgewählt werden oder es kann ein bestimmter Zeitraum in die Vergangenheit geschaut werden (bekannt aus v2).

Beim Chart-Typ kann zwischen Linie und Bar gewählt werden, wobei Bar die Säulen bei vielen Daten kaum erkennbar sind. Hierfür gibt es eine Aggregationsmöglichkeit, um auch die Performance zu erhöhen. Diese zeigt dann aber (logischerweise) weniger Daten / weniger Details an.

Die Legende kann nun ausgeblendet werden. Wird diese ausgeblendet, können die Datenpunkte identische Namen haben. Sofern diese eingeblendet ist und es Datenpunkte mit identischen Namen gibt, dann fügt jarvis (2) an, um diese zu unterscheiden.

Die Achsen können nun frei konfiguriert werden (siehe https://echarts.apache.org/en/option.html#xAxis bzw. https://echarts.apache.org/en/option.html#yAxis für alle Optionen). Dies inkludiert insbesondere min / max für die y-Achse.

Außerdem kann die Zeitangabe auf der X-Achse frei definiert werden.

@mcuiobroker
Copy link
Collaborator Author

Kann leider erst nächste Woche testen. Hab zurzeit kein System. Melde mich dann.

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Aug 20, 2021

v3.0.0-alpha.53
Bei Größenänderung von Gesamt-index.html wird der Graph nicht kleiner (bzw angepasst), er bleibt in seiner Ursprungsgröße.
image
Schaltet man danach wieder auf volle Größe, wird der Graph kleiner, auf die Größe, die vorher sinnvoll gewesen wäre, nur zu spät.

v3-historygraph-Größenänderung

Zeitweise (nicht immer reproduzierbar) entstehen auch an den Kurvenbeschreibungen zusätzliche Zahlen in Klammern, obwohl sich der Inhalt nicht ändert.
Ist auch im Video kurz zu sehen: Corona (1) (2) (3)

@Zefau Zefau removed ✔️ verify-fix #prio:HIGH ❗ Priority: HIGH :exclamation: labels Aug 20, 2021
@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Aug 20, 2021

Bekomme auch folgende Fehler angezeigt in der F12-Konsole:
image
Wann kann man denn den Pro-User anmelden? Ich kann sonst bestimmte Sachen gar nicht mehr testen?
Wenn die Notifications zu Pro gehören, dann sollte der Fehler in v2.2.3 auch korrigiert werden.
In v2.2.3 bekommt man auch einen Fehler in F12-Konsole angezeigt:
https://forum.iobroker.net/topic/30668/jarvis-v2-2-0-just-another-remarkable-vis/4868

@Zefau
Copy link
Owner

Zefau commented Aug 20, 2021

https://forum.iobroker.net/topic/30668/jarvis-v2-2-0-just-another-remarkable-vis/4868 ist kein Fehler in Verbindung mit Pro.
Vielmehr scheint hier das parsen der Notifications fehlzuschlagen, also jarvis.X.notifications ist kein JSON-Array.

In v3 ist Pro nun implementiert, aber noch nicht aktiviert. Eigentlich sollte alles wie normal funktionieren.
Ich schau mir das gleich mal an.

@Zefau Zefau removed this from the ROADMAP v3.0.0 milestone Sep 2, 2021
@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Sep 2, 2021

Mit v3.0.0-alpha.61 ist nur noch der Effekt mit den zusätzlichen Zahlen in Klammern aufgefallen:
Corona (1) (2) (3)

image

Vergrößerung und Verkleinerung von BrowserSeite auf dem PC passen.
Auf dem Handy nicht. Dort wird der HistoryGraph zu breit dargestellt. Im Querformat fehlen auf der rechten Seite ca 10%. Im Hochformat ca 50%.

Hochformat:
image
Querformat:
image

@Zefau Zefau added this to the BETA RELEASE milestone Sep 2, 2021
@Zefau
Copy link
Owner

Zefau commented Sep 26, 2021

besteht das Problem mit (1) noch mit v3.0.0-alpha.81 ?

@mcuiobroker
Copy link
Collaborator Author

Momentan werden die Daten gar nicht angezeigt. Ich würde damit warten, bis die anderen Sachen wieder funktionieren und dann nutze ich mal Prod-Daten.

@Zefau
Copy link
Owner

Zefau commented Sep 26, 2021

Das Hochformat / Querformat Problem sollte mit v3.0.0-alpha.82 behoben sein.

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Sep 26, 2021

Das Hoch-Querformat Verhalten tritt nicht mehr auf.
Problem mit der (1) auch nicht, da ich gerade nicht mehr weiß, wie man die unterschiedlichen Kurven einstellen kann.
image
Automatisch setzt er die zwei Kurven so:
image

EDIT: Musste nur die Label anpassen. Passt so Danke.
Problem bzgl Responsive auf dem Handy tritt auf, wenn eine jsonTable auf der Seite definiert ist. Dann werden die Element zu breit angesetzt und es fehlt die Hälfte auf dem Bildschirm.Dies ist aber in einem anderen Issue noch offen.
#953

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Sep 26, 2021

Von den einstellbaren Werten für yAxis (https://echarts.apache.org/en/option.html#yAxis) ziehen nur:
{"position":"right","name":"Anzahl"}
Liegt es am Pro Account?

Manchmal ist es nach einer Änderung und Neuladen der Seite so, dass dieser Zustand bestehen bleibt und man F5 drücken muss:
image
Nach F5 sind die Daten dann sofort da.

@Zefau
Copy link
Owner

Zefau commented Sep 27, 2021

Achsen sind nur mit Pro Account, der aber in der Alpha generell aktiviert ist. Es sollte also alles gehen.

Was versuchst du denn genau?

Das mit dem Neuladen kann sein, da er das gechached hat. Da kann ich nochmal schauen.

@mcuiobroker
Copy link
Collaborator Author

mcuiobroker commented Sep 27, 2021

Wollte Color, fontSize usw. testen und hatte übersehen, dass es in ein JSON nameTextStyle muss. Funktioniert.
Was ich noch nicht herausgefunden habe ist, wie kann man bei zwei unterschiedlichen Kurven die linke yAchse für die eine verwenden und die rechte yAchse für die andere. Da es ja vorkommt, dass eine Kurve eine Zahl zeigt und die andere z.B. %-Werte.

Beispiel, welches funktioniert:
{"position":"right","name":"Anzahl","nameTextStyle":{"color":"red","backgroundColor":"grey","fontSize":14}}

@Zefau
Copy link
Owner

Zefau commented Sep 27, 2021

Vom ursprünglichen Bug Report ist alles weg, richtig (mit Berücksichtigung von #953)? Offen ist nur die Frage bzgl. zweiter y-Achse?

@mcuiobroker
Copy link
Collaborator Author

Ja

@Zefau
Copy link
Owner

Zefau commented Sep 27, 2021

#326

@Zefau Zefau closed this as completed Sep 27, 2021
@Zefau
Copy link
Owner

Zefau commented Sep 27, 2021

kommt mit der nächsten alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working Module HistoryGraph Module HistoryGraph
Projects
None yet
Development

No branches or pull requests

2 participants