J’ai bien reçu ta DB @Psoy , je confirme que le bug est le même que rencontré par @lmilcent , à savoir qu’il y a des valeurs de capteurs qui sont enregistré en “NaN” en DB par l’intégration Zigbee2mqtt (et récemment)
@AlexTrovato Le fix qu’on avait fait n’a visiblement pas été assez, il y a toujours le soucis !
Je vais faire un correctif et ça partira dans la prochaine version de Gladys
Psoy
Mars 22, 2022, 5:01
82
Encore merci pour ton aide
Bonne soirée
J’ai trouvé un moyen de bétonner encore plus cette partie !
J’ai fais une PR ici :
GladysAssistant:master
← GladysAssistant:prevent-service-from-inserting-nan-numbers
opened 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!
1 « J'aime »
C’est fou !
J’espère que la validation “isFloat” ne rejette pas les entiers
On est sûr que @Psoy est sur une version “récente” ?
Psoy
Mars 22, 2022, 8:17
85
Bonsoir,
Oui la 4.8.1, depuis avant hier soir.
Ce capteur était bien historisé sous l’intégration Mqtt et là je l’ai passé sous zigbbe2Mqtt depuis 4 jours et plus d’historique.
Non! Je suis allé voir le code de la lib de validation qu’ils utilisent, et dans les tests les entiers sont bien acceptés
Le bug est bien présent, actuellement si tu envoyais un texte ça passait !
Pour information @Psoy , j’ai mergé mon fix ( Make device state validation more robust (#1480) · GladysAssistant/Gladys@dc5a183 · GitHub ) qui partira dans la prochaine release de Gladys.
Je pense faire une release début de semaine prochaine avec ce correctif de bug ( pas aujourd’hui, c’est interdit de deploy le vendredi )
2 « J'aime »
Psoy
Mars 25, 2022, 2:16
88
Merci Pierre-Gilles, je patiente pas de probleme
je connais ça le vendredi, on range les ouitls
1 « J'aime »
@Psoy Comme prévu, nouvelle version de Gladys aujourd’hui avec le correctif !
J’espère que ça va résoudre le souci chez toi
Salut à tous !
C’est lundi, pas de temps à perdre, c’est l’heure d’une nouvelle version de Gladys avec un ensemble de correctifs
Les correctifs
Les capteurs de Co2 affichent désormais une couleur en fonction de la concentration en Co2 dans l’air, selon les recommandations :
[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
Mars 28, 2022, 12:23
90
je viens de vérifier, mais la 4.8.2 n’est pas encore arivvée jusqu’ a moi . je te confire ça des que possible
merci encore
1 « J'aime »
Psoy
Mars 29, 2022, 10:19
91
ça y est la 4.8.2 est passée et le problème est résolu
Merci encore
2 « J'aime »