Ich habe deine DB erhalten @Psoy , ich bestätige, dass der Bug derselbe ist wie der, den @lmilcent hatte, nämlich dass es Sensorwerte gibt, die in der DB als „NaN“ durch die Zigbee2mqtt-Integration (und kürzlich) gespeichert werden.
@AlexTrovato Der Fix, den wir gemacht haben, hat offensichtlich nicht ausgereicht, das Problem besteht immer noch!
Ich werde einen Korrektur machen und das wird in der nächsten Version von Gladys enthalten sein
Psoy
22. März 2022 um 17:01
82
Nochmals vielen Dank für deine Hilfe
Ich habe einen Weg gefunden, diesen Teil noch weiter zu zementieren!
Ich habe einen PR hier gemacht:
master ← prevent-service-from-inserting-nan-numbers
ouvert 05:11PM - 22 Mar 22 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](https://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!
Das ist verrückt!
Sind wir sicher, dass @Psoy eine « aktuelle » Version verwendet?
Psoy
22. März 2022 um 20:17
85
Guten Abend,
Nein! Ich habe mir den Code der Validierungsbibliothek angesehen, die sie verwenden, und in den Tests werden ganze Zahlen akzeptiert
Der Fehler ist tatsächlich vorhanden, aktuell würde Text durchgehen!
Zur Information @Psoy , ich habe meinen Fix (Make device state validation more robust (#1480) · GladysAssistant/Gladys@dc5a183 · GitHub ) gemerged, der in der nächsten Gladys-Release enthalten sein wird.
Ich denke, ich werde Anfang nächster Woche eine Release mit diesem Bugfix durchführen (nicht heute, am Freitag ist es verboten zu deployen :D).
Psoy
25. März 2022 um 14:16
88
Danke Pierre-Gilles, ich warte nicht, kein Problem
Das kenne ich, am Freitag räumt man die Werkzeuge weg
@Psoy Wie erwartet, heute neue Version von Gladys mit dem Fix!
Ich hoffe, das löst dein Problem
Salut à tous !
C’est lundi, pas de temps à perdre, c’est l’heure d’une nouvelle version de Gladys avec un ensemble de correctifs [Indice-PPM-1-1024x538]
Correction d’un bug où l’intégration Zigbee2mqtt créait des valeurs de capteurs au format “texte” dans la base de donnée (au lieu de flottants), ce qui créait des bugs sur l’agrégatio…
Psoy
28. März 2022 um 12:23
90
Ich habe gerade nachgeschaut, aber die 4.8.2 ist noch nicht bei mir angekommen
Psoy
29. März 2022 um 10:19
91
Jetzt ist die Version 4.8.2 da und das Problem ist gelöst. Nochmals vielen Dank.
. Ich bestätige dir das, sobald es möglich ist.
