search

tthma / homebridge-linky-enedis-meter

MIT License
8 stars 1 forks source link

chore(enedis-call): Improve error and handle with async/await #11

Open Hydrog3n opened 1 year ago

Hydrog3n commented 1 year ago

Hello, I improved a little the code. The error from linky is now only the error message. We can see the full message with the debug activated in homebrigde

image

tthma commented 1 year ago

J'ai déjà fait en async. Le problème c'est que si le temps de loop est trop court le GC n'a pas le temps de détruire les objets et ca fait exploser la mémoire et fini par crasher. De plus je suis pas sûr que fakegato gère le async en ecriture sur la mémoire tampon ca risque de mettre le bazar. A voir je vais attendre un peu d'avoir votre retour si y a des crash des ratés avant de merge le code et de faire la mise a jour

Hydrog3n commented 1 year ago

Oui j'ai juste utiliser une différente façon d’écrire les async c'est plus lisible de mon coté ça tourne depuis 2j

Clairement le but c'est de reduire le code et l'impact mémoire à la fin pour évité que la heap memory soit saturé.

Hydrog3n commented 1 year ago

Si jamais on peut essayé de faire une beta pour deployer sur nos homebridge.

Je regarde aussi pour gérer la heap memory est ne pas lancer les queries si pas assez.

tthma commented 1 year ago

Si jamais on peut essayé de faire une beta pour deployer sur nos homebridge. ca je sais pas comment on fait je suis nouveau sur github en général je développe des trucs pour moi tout seul. Je regarde aussi pour gérer la heap memory est ne pas lancer les queries si pas assez. Normalement y a une variable qui empêche de relancer une querry en tout début de la fonction. Mais les essaies que j'avais fait n'étais pas concluant.

De plus je galère à debuger chez moi bouygue me bloque l'accès au site d'enedis va savoir pourquoi je le fait en partage de co sur mon tel.