# Help! Problème avec l'agrégation?

**URL:** <https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630>\
**Category:** Configuration\
**Created:** [Octobre 25, 2021, 5:54 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630 "2021-10-25T17:54:08Z")\
**Posts on this page:** 11\
**Page:** 5

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Mars 22, 2022, 4:56 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/81 "2022-03-22T16:56:53Z")

</div>

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 !

 ![Screenshot 2022-03-22 at 16.54.15](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/4/4091465e7996fbd230b7885867bc7090e0ecd22e.png)

Je vais faire un correctif et ça partira dans la prochaine version de Gladys 🙂

---

<div class="post-metadata">

**Author:** ![Psoy](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/psoy/32/9005_2.png) [@Psoy](https://community.gladysassistant.com/u/Psoy)\
**Post date:** [Mars 22, 2022, 5:01 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/82 "2022-03-22T17:01:55Z")

</div>

Encore merci pour ton aide  
Bonne soirée

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Mars 22, 2022, 5:12 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/83 "2022-03-22T17:12:35Z")

</div>

J’ai trouvé un moyen de bétonner encore plus cette partie !

J’ai fais une PR ici :

> <https://github.com/GladysAssistant/Gladys/pull/1480>
>
> \### 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!

---

<div class="post-metadata">

**Author:** ![AlexTrovato](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/alextrovato/32/10705_2.png) [@AlexTrovato](https://community.gladysassistant.com/u/AlexTrovato)\
**Post date:** [Mars 22, 2022, 8:11 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/84 "2022-03-22T20:11:07Z")

</div>

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” ?

---

<div class="post-metadata">

**Author:** ![Psoy](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/psoy/32/9005_2.png) [@Psoy](https://community.gladysassistant.com/u/Psoy)\
**Post date:** [Mars 22, 2022, 8:17 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/85 "2022-03-22T20:17:21Z")

</div>

> [@AlexTrovato](#):
>
> On est sûr que @Psoy est sur une version “récente” ?

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.

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Mars 23, 2022, 5:58 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/86 "2022-03-23T17:58:22Z")

</div>

> [@AlexTrovato](#):
>
> J’espère que la validation “isFloat” ne rejette pas les entiers 😃

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 🙂

 ![Screenshot 2022-03-23 at 17.57.16](https://gladys-assistant-community.b-cdn.net/uploads/default/original/2X/a/abd301f3cf1a712c87b7e20f5c87b0e42499916a.png)

> [@AlexTrovato](#):
>
> On est sûr que @Psoy est sur une version “récente” ?

Le bug est bien présent, actuellement si tu envoyais un texte ça passait !

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Mars 25, 2022, 2:01 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/87 "2022-03-25T14:01:38Z")

</div>

Pour information @Psoy, j’ai mergé mon fix ( [Make device state validation more robust (#1480) · GladysAssistant/Gladys@dc5a183 · GitHub](https://github.com/GladysAssistant/Gladys/commit/dc5a18378d258b8d1f58f5f89f3411a7a1443f53) ) 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 😃 )

---

<div class="post-metadata">

**Author:** ![Psoy](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/psoy/32/9005_2.png) [@Psoy](https://community.gladysassistant.com/u/Psoy)\
**Post date:** [Mars 25, 2022, 2:16 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/88 "2022-03-25T14:16:50Z")

</div>

Merci Pierre-Gilles, je patiente pas de probleme

> [@pierre-gilles](#):
>
> pas aujourd’hui, c’est interdit de deploy le vendredi 😃 )

je connais ça le vendredi, on range les ouitls 😆

---

<div class="post-metadata">

**Author:** ![pierre-gilles](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/pierre-gilles/32/5886_2.png) [@pierre-gilles](https://community.gladysassistant.com/u/pierre-gilles)\
**Post date:** [Mars 28, 2022, 12:19 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/89 "2022-03-28T12:19:53Z")

</div>

@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 🙂

> [@Gladys Assistant v4.8.2 est disponible, avec un ensemble de correctifs 🛠️](https://community.gladysassistant.com/t/gladys-assistant-v4-8-2-est-disponible-avec-un-ensemble-de-correctifs/7107):
>
> Salut à tous ! C’est lundi, pas de temps à perdre, c’est l’heure d’une nouvelle version de Gladys avec un ensemble de correctifs smiley Les correctifs Les capteurs de Co2 affichent désormais une couleur en fonction de la concentration en Co2 dans l’air, selon les recommandations : 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…

---

<div class="post-metadata">

**Author:** ![Psoy](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/psoy/32/9005_2.png) [@Psoy](https://community.gladysassistant.com/u/Psoy)\
**Post date:** [Mars 28, 2022, 12:23 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/90 "2022-03-28T12:23:34Z")

</div>

> [@pierre-gilles](#):
>
> J’espère que ça va résoudre le souci chez toi 🙂

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

---

<div class="post-metadata">

**Author:** ![Psoy](https://gladys-assistant-community.b-cdn.net/user_avatar/community.gladysassistant.com/psoy/32/9005_2.png) [@Psoy](https://community.gladysassistant.com/u/Psoy)\
**Post date:** [Mars 29, 2022, 10:19 UTC](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630/91 "2022-03-29T10:19:58Z")

</div>

> [@pierre-gilles](#):
>
> J’espère que ça va résoudre le souci chez toi 🙂

ça y est la 4.8.2 est passée et le problème est résolu  
Merci encore

[Page précédente](https://community.gladysassistant.com/t/help-probleme-avec-lagregation/6630.md?page=4)
