Skip to content

Ajout d'un éditeur visuel#22

Draft
pa-martin wants to merge 4 commits into
Master13011:mainfrom
pa-martin:main
Draft

Ajout d'un éditeur visuel#22
pa-martin wants to merge 4 commits into
Master13011:mainfrom
pa-martin:main

Conversation

@pa-martin
Copy link
Copy Markdown

@pa-martin pa-martin commented May 15, 2026

Bonjour,

Je suis tombé sur ce repo un peu par hasard en voulant voir ce qui existait sur HACS en lien avec l'api formidable de la SNCF. J'ai tout de suite accroché à la carte lovelace proposée même si j'y voyais quelques défauts pour lesquels je me permets de vous proposer des solutions :

  • le manque d'un éditeur visuel
  • l'absence de l'heure de départ
  • l'étape de récupération de l'identifiant du device

La solution fonctionne dans un cas nominal (cf. capture ci-dessous). Je testerais dans la journée des cas un peu plus orthodoxe (situations de retards, trains annulées, ect...).
Je me suis aussi permis d'améliorer un peu la qualité de code et j'ai ajouté de la documentation sur les méthodes (GitHub Copilot℗ l'a en grande partie écrite : il peut y avoir des incohérences).
Il se peut que la PR rentre en conflit avec la PR#18 de ProBreizh35. À voir comment est-ce qu'on s'aligne...

TODO avant PR final :

  • Tester l'affichage d'un train en retard, surtout sur les heures initiales et réelles
  • Tester plus en profondeur l'option "animation_duration" et faire en sorte de rendre plus dynamique la progression de la barre de manière à ce que le paramètre définisse uniquement le délai avant le départ où l'icône du train apparait.

Autres améliorations possibles :

  • la gestion des icônes HA de manière plus générique que simplement les emoji "textuel" (ex : mdi:home)
  • certains trains passant par les gares de départ et d'arrivée semblent être absents -> à approfondir
image image image

@ProBreizh35
Copy link
Copy Markdown

ProBreizh35 commented May 16, 2026

Salut, et bien écoute, j'aime bien ton idée, si tu veux je te laisse t'adapter à ce que j'est mis en place ?
Si tu arrive à régler le bug, du non affichage de la carte depuis l'extérieur çà serai top, j'ai beau chercher, avoir ajouter la card en variable local, rien n'y fait.

Par contre j'ai l'impression que ta pas les bon fichier. J'ai rajouter les gare intermédiaire (false par default) la vitesse réel du déplacement du train les changement d'horaire en cas de retard, le nombres de train entre 1 et 20 (5 par default), la cause des retard (true/false).
Dans le futurs je compte ajouter les voie de départ et d'arrivé(activable ou non via un true/false)

Jolie boulot !

On devrai faire une branche dev afin que l'on est les même fichiers de travail tu ne pense pas?
Je reste à ton écoute :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants