Für mich gibt es zwangsläufig zwei Zeitpunkte: den „historischen“ Punkt und die aktuelle Uhrzeit
Da wir über einen Zustand sprechen, muss man zwangsläufig die letzte Linie zwischen dem letzten Punkt und jetzt ziehen, oder?
Ich habe die Änderung in diesem Sinne vorgenommen.
Ich habe (nur für die Binärdateien) hinzugefügt:
Es ist gepusht
Hallo @pierre-gilles ,
Wie oben erwähnt, habe ich die Änderungen so vorgenommen, wie du es dir vorgestellt hast.Add binary chart by Terdious · Pull Request #2094 · GladysAssistant/Gladys · GitHub
Hallo @Terdious
Ich habe dir eine funktionale Review zum Thema gemacht:
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
Noch keine technische Review, solange es nicht funktional in Ordnung ist!
Hey @pierre-gilles ,
Ich habe normalerweise die Anmerkungen angewendet und überprüft, außer:
Ich habe versucht, es nachzubilden, aber trotz Zoom rein/raus und Auflösung ändern, etc. habe ich nie dein Ergebnis erreicht. Allerdings hatte ich vor meiner letzten Änderung das gleiche Problem, aber ich hatte es vor deiner Überprüfung korrigiert. Deshalb habe ich trotzdem auf maximal 10 Zeichen reduziert, nur für den Fall…^^
Und für den Power Plug habe ich dir auf GitHub geantwortet, aber ich kann es nicht reproduzieren. Ich habe ein Log für die Einheiten hinzugefügt, um mehr zu sehen.
Danke für die Korrekturen!
Terdious:
Ich habe versucht, es zu reproduzieren, aber trotz Zoomen/Herauszoomen, Auflösung ändern usw. habe ich nie dein Ergebnis erreicht. Allerdings hatte ich vor meiner letzten Änderung dasselbe Problem, aber ich habe es vor deiner Überprüfung behoben. Also habe ich es trotzdem auf maximal 10 Zeichen reduziert, nur für den Fall…^^
Okay, ich kann es auch nicht reproduzieren!
Ich habe dir einen Kommentar für den Rest hinterlassen:
master ← Terdious:add-binary-chart
@Terdious Thanks for the fixes!
Here is the payload that causes the freeze: …
```
{"id":"e499dd5e-6482-4cbd-a73e-159d53c9b402","name":"Graph","selector":"graph","type":"main","visibility":"private","user_id":"275faa00-8a9c-4747-8fbe-417ddb966b16","created_at":"2024-06-10T08:20:20.784Z","updated_at":"2024-06-17T11:30:17.089Z","boxes":[[{"type":"chart","device_features":["mqtt-priseonoff"],"units":[null],"title":"Binaire","chart_type":"","interval":"last-day","display_axes":true}],[],[]]}
```
The response is 422 :
```
{
"status": 422,
"code": "UNPROCESSABLE_ENTITY",
"properties": [
{
"message": "\"[0][0].chart_type\" is not allowed to be empty",
"attribute": "boxes",
"value": [
[
{
"type": "chart",
"device_features": [
"mqtt-priseonoff"
],
"units": [
null
],
"title": "Binaire",
"chart_type": "",
"interval": "last-day",
"display_axes": true
}
],
[],
[]
],
"type": "Validation error"
}
]
}
```
It's just because I didn't select the "chart type" ^^ It's maybe not related to this PR, but I wonder if we could make a fix to avoid freezing in this situation...
I have another feedback, now the date in english is not correct:
Hallo @pierre-gilles ,
Passt dir das für deinen letzten Kommentar zu dieser PR:
Ich glaube, das war der letzte Punkt, der behandelt werden musste.
Echt sehr cool
Ich habe dir 2 Rückmeldungen gegeben.
Ich denke, wir können danach einen Cloudflare Pages Build machen, damit Gladys Plus Nutzer mit ihrer Instanz testen können, da es nur Frontend ist, sollte es funktionieren (wenn die Nutzer ihren „binären“ Graphen nach dem Test löschen).
Ok gut für mich (ich habe dir auch dort unten geantwortet)
Aber bei den WebSockets gibt es bei mir keine Probleme, ich bin 2 Minuten auf der Seite geblieben und die Aktualisierungen erfolgen alle 22s/30s ohne Seitenneuladung … Allerdings wird die Popup nicht automatisch aktualisiert, man muss die Maus bewegen.
pierre-gilles:
Ich denke, wir können später einen Cloudflare Pages-Build machen, damit Gladys Plus-Nutzer mit ihrer Instanz testen können, da es nur Frontend ist, sollte es funktionieren (wenn die Nutzer ihren „binären“ Graphen nach dem Test löschen)
Ah ja, das wäre wirklich super!!^^
Danke für die schnelle Änderung! Ich habe dir unten geantwortet, für mich ist beides in Ordnung.
Ich habe einen PR für die Cloudflare Pages Builds erstellt (behalte deinen PR):
master ← Terdious-add-binary-chart
ouvert 02:36PM - 30 Aug 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)
- [ ] Is the linter passing? (`npm run eslint` on both front/server)
- [ ] 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.
- [ ] 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
Please provide a description of the change here. It's always best with screenshots, so don't hesitate to add some!
Die URL sollte in ein paar Minuten bereit sein.
Super !!
@guim31 , falls du uns liest … ^^
Das ist hier verfügbar:
https://terdious-add-binary-chart.gladys-plus.pages.dev
Ich empfehle, es auf einem separaten Dashboard zu testen und das Dashboard nach dem Test zu löschen (um die eigene Produktion nicht zu verschmutzen).
guim31
30. August 2024 um 15:07
75
So sieht es bei mir aus:
Ich hätte meine Funktion gerne umbenennen können (wie bei Gerät des Raums), denn so ist es einfach zu lang ^^
Ansonsten ist bei mir alles gut !!
Danke für den Test, Guim31!!
Am Ende wundert mich das nicht.
@Terdious ich weiß nicht, ob du mit den Optionen der y-Achse gespielt hast?
Typischerweise kannst du eine minWidth festlegen
Aber klar, das wird dann auf dem Diagramm Platz wegnehmen, kein Geheimnis, solche Diagramme würden stark von der Möglichkeit profitieren, 2 oder 1 Spalten zu haben ^^
Ansonsten sollte dem Benutzer die Möglichkeit gegeben werden, das Gerät umzubenennen, wie beim Widget „Geräte“
pierre-gilles:
Typischerweise kannst du eine minWidth angeben
Danach, okay, das wird natürlich auf dem Diagramm Platz wegnehmen, kein Geheimnis, diese Art von Diagramm wird stark von der Möglichkeit profitieren, 2 oder 1 Spalten zu haben ^^
Ja, und genau das war es, es reduzierte das Diagramm so sehr, dass ich es aufgegeben habe. Aber tatsächlich kann eine Mischung aus beidem die extremen Fälle bewältigen.
Ich gehe davon aus? ‹ minWidth › + Möglichkeit zur Umbenennung (das Bearbeitungsfeld wird riesig sein
Ich wäre auch interessiert, das zu sehen! Ja, das wird ein riesiges Bearbeitungsfeld
Ich habe einen ersten Kompromiss gefunden, den ich interessant finde, mit ‹ minWidth › und einigen Änderungen. Ich habe eine der Boxen voll ausgelastet, um sie wirklich zu testen.
Ich habe gerade gepusht, @pierre-gilles startet Cloudflare automatisch neu oder ist das eine manuelle Aktion von dir?
Ich arbeite weiter an der Möglichkeit zum Umbenennen, also wenn es eine Aktion von dir ist, ist es vielleicht nicht nötig, ein Bild vorher neu zu starten. Ich sage dir Bescheid.
)
