Fehler 404 klingt gut für mich!
Es ist in Arbeit !!
Die Leistung ist mit DuckDB völlig anders:
1 Stunde = 184 Zustände
Wechsel zu 24h = 1560 Zustände
Wechsel zu 7 Tagen = 3700 Zustände
Wechsel zu 3 Monaten = 37300 Zustände (gut, aber dann wird die Navigation auf diesem Stand) und wenn mehrere Geräte, dann wird es heiß.
Ich schlage vor:
24h und weniger keine Begrenzung der Punkte
Darüber hinaus: 4000 Punkte / Anzahl der zu zeigenden Features
Beim Zoomen neu berechnen, um immer genauer zu werden
Ein Dropdown-Menü zur Auswahl für die Anzeige nach Aggregat (wie bei Metabase)
Das ist kein Muss!
Terdious:
24h und weniger: keine Punktebegrenzung
Darüber hinaus: 4000 Punkte / Anzahl der anzuzeigenden Features
Warum nicht einfach überall eine Grenze setzen? 4000 Punkte erscheinen mir allerdings übertrieben, aktuell werden im Dashboard 200 angezeigt
Yes! aber immer mit der gleichen Grenze
Eigentlich muss man einfach genau die gleiche Abfrage wie im Dashboard beibehalten, aber mehr Punkte anfordern und die Möglichkeit haben, einen Zeitraum und nicht nur eine Dauer zu spezifizieren
Yes, auf jeden Fall!
Parfait
Oki, ich sehe, wie es mit 1000 Punkten aussieht.
pierre-gilles:
Das ist kein Muss!
Oki, dann sehen wir uns das später an ^^
guim31:
Ja, das hatte ich verstanden, aber ich bin trotzdem ungeduldig. Eigentlich finde ich, dass der Anzeigeteil der Diagramme vor allem die Möglichkeit benötigte, sie in einer größeren Größe anzuzeigen, und das ist für mich jetzt perfekt!
Zwischen dem und den binären Sensoren bin ich von diesen letzten Entwicklungen begeistert!!
PR zur Auswahl der Anzahl der Spalten im Dashboard im Test von Gladys Plus unter https://terdious-add-choice-number-col.pages.dev/ für die Interessierten!!^^
Und schließlich die erweiterte Ansicht der Diagramme:https://terdious-add-expand-charts.pages.dev/
Hallo @Terdious und danke für all deine PRs
Ich habe heute Morgen eine Reihe von Reviews durchgeführt:
Ich habe deine PR zur Anzahl der dynamischen Spalten im Dashboard gemerged:
master ← Terdious:add-choice-number-columns-dashboard
ouvert 08:32PM - 03 Jun 24 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- ~~If your changes affects code, did your write the tests?~~
- ~~Are tests passing? (`npm test` on both front/server)~~
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Adding the deletion or addition of columns to the dashboard
Kurzes Feedback zu deiner PR, die die Größe der Diagramme anpasst:
master ← Terdious:improvement-height-charts
ouvert 01:57PM - 05 Sep 24 UTC
…d on additional features
### Pull Request check-list
To ensure your Pull … Request can be accepted as fast as possible, make sure to review and check all of these items:
- ~~If your changes affects code, did your write the tests?~~
- ~~Are tests passing? (`npm test` on both front/server)~~
- [x] Is the linter passing? (`npm run eslint` on both front/server)
- [x] Did you run prettier? (`npm run prettier` on both front/server)
- ~~If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)~~
- ~~Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.~~
- [x] If your changes modify the API (REST or Node.js), did you modify the API documentation? (Documentation is based on comments in code)
- ~~If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).~~
- ~~Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).~~
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Small PR to increase the height of the charts depending on the number of features selected. Currently the curve is squashed to make room for the legend and contain the size of the box.
Bei der PR zu den Binärgeräten habe ich einige Rückmeldungen, aber nichts Ernstes.
master ← Terdious:add-binary-chart
ouvert 11:44AM - 03 Jun 24 UTC
### Pull Request check-list
To ensure your Pull Request can be accepted as fa… st as possible, make sure to review and check all of these items:
- [x] If your changes affects code, did your write the tests?
- [x] Are tests passing? (`npm test` on both front)
- [x] Is the linter passing? (`npm run eslint` on both front)
- [x] Did you run prettier? (`npm run prettier` on both front)
- [x] If you are adding a new features/services, did you run integration comparator? (`npm run compare-translations` on front)
- [x] Did you test this pull request in real life? With real devices? If this development is a big feature or a new service, we recommend that you provide a Docker image to the community ([french forum](https://community.gladysassistant.com/)/[english forum](https://en-community.gladysassistant.com/)) for testing before merging.
- [ ] If you are adding a new features/services which needs explanation, did you modify the user documentation? See [the GitHub repo](https://github.com/GladysAssistant/v4-website) and the [website](https://gladysassistant.com).
- [ ] Did you add fake requests data for the demo mode (`front/src/config/demo.js`) so that the demo website is working without a backend? (if needed) See [https://demo.gladysassistant.com](https://demo.gladysassistant.com).
NOTE: these things are not required to open a PR and can be done afterwards / while the PR is open.
### Description of change
Taken from the original PR: #Binary graph #1948
Thanks to @callemand for all the work done
Nochmals danke für diese PRs, die eindeutig vielen Leuten gefallen werden, mir als Erstem
Zögere nicht, wenn du Fragen zu den Rückmeldungen hast
Hallo @pierre-gilles , keine Fragen, danke für die Reviews.
zedyxer
4. Oktober 2024 um 09:08
169
Ein riesiges Dankeschön für eure Arbeit, Jungs ^^
@Terdious Danke für dein Feedback
Ich habe den PR für die Binärdiagramme gemerged! Bei dem anderen PR gibt es jetzt einen Konflikt mit den Binärdiagrammen, weil es dasselbe betrifft. Kannst du den Konflikt bitte beheben?
Edit: Build von Gladys Assistant 4.46.0 läuft gerade. In 4 Stunden veröffentlicht!
https://github.com/GladysAssistant/Gladys/actions/runs/11208245077
Top!!
Konflikt behoben und getestet.
! Im Bau (nicht vollständig funktionsfähig und Design noch nicht endgültig). Ermöglicht einen Eindruck der laufenden Entwicklung. Ich habe es in den automatischen Build gestellt, daher wird es im Laufe der Zeit aktualisiert.
