search

kaloskagatos / EDF-Tempo-E-Ink-Display

GNU General Public License v3.0
9 stars 0 forks source link

RETRY_DELAY_MIN RETRY_COUNT #4

Open bucolo opened 9 months ago

bucolo commented 9 months ago

Bonjour, j'utilise la version avec le preview RTE. sur home assistant avec le module RTE tempo j'ai remarqué que la couleur du lendemain "arrivait" souvent entre 6h30 et 6h40 , des fois un peu avant , des fois un peu après. j'ai donc modifié l'heure de réveil à 6h30 , reglé RETRY_DELAY_MIN = 5 et RETRY_COUNT = 7 pour avoir des essais de 6h30 a 7h05 . Mais hélas ca n'a pas l'air de fonctionner, ce matin la couleur rouge est arrivé à 6h32 et a 6h46 l'esp m'affichait toujours ??? . Et à 6h45 je n'ai pas vu de réveil de l'esp. j'ai fais un reset juste après le rouge s'est bien affiché. J'ai sans doute zappé un truc dans le fonctionnement des retry , merci de m’éclairer.

LArtisanDuDev commented 9 months ago

Je n'ai pas géré le retry sur l'API RTE. Il faudrait rajouter un booleen du genre isRTEFound et adapter le test de la ligne https://github.com/LArtisanDuDev/EDF-Tempo-E-Ink-Display/blob/main/eTempo/eTempo.ino#L557 La subtilité c'est que le retry sur l'API RTE n'est pas nécessaire à minuit et à 11h...

Edit : j'ai codé vite fait un truc, mais du coup ça embarque pas mal de modifs que j'avais en cours. Sur la branche : https://github.com/LArtisanDuDev/EDF-Tempo-E-Ink-Display/tree/platformio => Passage à vscode et platformio au lieu de Arduino IDE. => Changement de l'affichage des jours restants. Ma MOA familiale voulait afficher le nombre de jours déjà effectués. => Suprression de la librairie Preferences qui écrit en NVS qui a un nombre de cycle d'écritures limités (bon 10000 on était super larges mais bon....). Passage en mémoire RTC qui est préservée au deepsleep. => Retry sur preview RTE.

Codé sur la pause de midi. A considérer en béta jusqu'au test de demain matin.

LArtisanDuDev commented 9 months ago

Qq petites corrections réalisées, ça devrait être ok ici : https://github.com/LArtisanDuDev/EDF-Tempo-E-Ink-Display/tree/platformio

bucolo commented 9 months ago

Super , merci, je viens de faire l'update (en gardant Arduino IDE) . On verra demain ou apres demain ce que ca donne. Merci encore ps: pour info j'ai testé, sur un autre lilygo T5, un ecran e-ink blanc noir rouge en remplacement de l’écran d'origine defectueux (le brochage est compatible). ca marche mais le rafraichissement d'ecran est long. Il faut prendre la bibliothèque GxEPD2. image Pour tester j'ai fais comme si le bleu était rouge ...

LArtisanDuDev commented 9 months ago

Désolé, mais je n'ai mis en place les évolutions que sur la branche platformio (qui compile environ 10 fois plus vite). Je n'ai pas mis à jour la branche main qui est pour Arduino IDE. Vu le compte de jours que vous avez sur votre photo vous devez avoir l'ancienne version. J'essaierai de mettre à jour la version Arduino IDE demain.

LArtisanDuDev commented 9 months ago

https://github.com/LArtisanDuDev/EDF-Tempo-E-Ink-Display/tree/main à jour (Arduino IDE)

bucolo commented 9 months ago

Merci c'etait bon, j'avais bien renommé le cpp en ino . C'est juste que pour la version "couleur" j'avais commencé les modifs depuis une ancienne version. Sinon j 'ai remarqué que le triangle s'affiche a 00:05 alors qu'a cette heure c'est normal. J'ai pu modifié ça . J'ai remis aussi le jours restant plutôt que les jours écoulé (ma femme préfère) mais en supprimant le /300 /43 /22 ce qui me permet de liberer de la place et d'afficher l'heure du prochain réveil.
image le boitier à imprimer : https://www.thingiverse.com/thing:4055993

LArtisanDuDev commented 9 months ago

Nos MOA décident de tout :) Le boitier peut être facilement rendu plus profond ? ma batterie est une 18650 qui est un peu épaisse...

bucolo commented 9 months ago

Plus épaisse et un peu plus large a priori. Il y a quelques boitier pour T5 2.13 sur thingverse mais aucun n'a l'air d'aller. Si tu me donnes les dimensions de ton support de 18650 mon fils devrait pouvoir recréer un stl à partir de celui que j'ai utilisé .

bucolo commented 9 months ago

Mes modifs concernant le retry pour la couleur du lendemain n'etait pas suffisante. Contrairement a ce que l'on pourrait penser il faut indiquer :

// Tableau des heures de réveil const WakeupTime wakeupTimes[] = { {0, 5, false}, // Réveil à 00:05 {6, 30, false}, // Réveil à 06:30 pour préview RTE {11, 5, true} // Réveil à 11:05 };

pour ne pas avoir de retry a 00:05