delphiki / hass-pronote

Pronote integration for Home Assistant
58 stars 9 forks source link

timetable_period exceed maximum size of 16384 bytes #54

Open tdelaux opened 10 months ago

tdelaux commented 10 months ago

Bonjour, il y a l'avertissement suivant qui remonte dans mes logs. J'ai l'impression que le "timetable_period" est un peu long. Il faudrait peut-être réduire la période remontée à 10 jours (ou 7 ou 12...).

Logger: homeassistant.components.recorder.db_schema
Source: components/recorder/db_schema.py:578
Integration: Recorder ([documentation](https://www.home-assistant.io/integrations/recorder), [issues](https://github.com/home-assistant/core/issues?q=is%3Aissue+is%3Aopen+label%3A%22integration%3A+recorder%22))
First occurred: 6 novembre 2023 à 16:43:05 (116 occurrences)
Last logged: 06:45:24

State attributes for sensor.pronote_xxx_alice_timetable_period exceed maximum size of 16384 bytes. This can cause database performance issues; Attributes will not be stored
State attributes for sensor.pronote_xxx_sixtine_timetable_period exceed maximum size of 16384 bytes. This can cause database performance issues; Attributes will not be stored
delphiki commented 10 months ago

Hello,

Yes, j'ai prévu de rendre le paramètre qui gère cette durée paramétrable, ça évitera les erreurs.

GaPhi commented 10 months ago

Plutôt que paramétrable, autant le rendre autoadaptatif (tronquer proprement, à la journée, avant la limite fatidique)

Le mar. 7 nov. 2023 à 08:43, ju @.***> a écrit :

Hello,

Yes, j'ai prévu de rendre le paramètre qui gère cette durée paramétrable, ça évitera les erreurs.

— Reply to this email directly, view it on GitHub https://github.com/delphiki/hass-pronote/issues/54#issuecomment-1797971642, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADUNWNIRIFHQO7C7WQEP2RLYDHRDDAVCNFSM6AAAAAA7AT26M6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJXHE3TCNRUGI . You are receiving this because you are subscribed to this thread.Message ID: @.***>

delphiki commented 10 months ago

On pourrait le faire, mais uniquement dans le sensor. Dans le coordinator, on aurait toujours X jours chargés, et dans le sensor, on va build dynamiquement les attributs pour ne pas dépasser la taille voulue. Ça fait un peu système D, mais je regarderai comment faire au mieux.

MickaelDENIS commented 9 months ago

Bonsoir, J'ai le même soucis, et en plus je pense que cela bloque le chargement des autres propriétés, car depuis que j'ai ce pb seul 8 entitées sont dispo

bckthomas commented 7 months ago

J'ai le même souci que @MickaelDENIS Depuis que j'ai ce message dans mes logs, les capteurs suivants sont indisponibles : Voici la liste des capteurs qui remontent en « indisponible »:

vingerha commented 6 months ago

le 'soucis' vient (un peu) de ma part. J'ai besoin de les voir dans les 2 semaines futurs pour planifier autre rdv. Ça donne tant des infos détaillés dans le sensor, peut-être pas toujour importants. Je vais voir comment faire mieux aussi

silversmarty commented 4 months ago

Bonjour, j'ai également ce bug : comment le résoudre en limitant le nombre de données retournées ? Merci beaucoup :-)