Ajout d'un éditeur visuel

[pa-martin]

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

[ProBreizh35]

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 :)

[pa-martin]

Si tu arrives à régler le bug, du non-affichage de la carte depuis l'extérieur ça serai top, j'ai beau chercher, avoir ajouter la card en variable local, rien n'y fait.

Done via la PR #23 🥵

On devrait faire une branche dev afin que l'on ait les même fichiers de travail tu ne penses pas?

Faire une branche de dev afin de tester l'intrégration via HACS avant de release une version stable est en effet une bonne chose. En revanche, je ne pense pas que travailler sur une branche commune soit une bonne idée. Les PR comme celle-ci permettent justement de relire le code de chacun sans que les idées des uns et des autres n'impactent leur propre développement.

Autrement dit : idéalement 1 PR = 1 idée / 1 développement, simplifiant ainsi son intégration, sa relecture et son implémentation.

Par contre, j'ai l'impression que tu n'as pas les bon fichier.
si tu veux je te laisse t'adapter à ce que j'ai mis en place ?

Dans ton cas, ta PR est assez "lourde", tu modifies énormément de fichier "core" (python) ainsi que le composant frontend. Si je me base sur tes développements et que ceux-là ne sont pas validés, avec mes propres évolutions ne pourront pas l'être non plus. Je te propose donc 2 solutions :

1. Ta PR est validée en première => Je pull tes modifications et j'adaptes les miennes à ton travail.
2. Ma PR est validée en première => J'adapte ton développement à ce que j'ai fait, de manière à ce que cela ne gène pas ton travail (la gestion des conflits pouvant être un poil technique pour un néophyte de Git).

Dans le futur, je compte ajouter les voies de départ et d'arrivée (activables ou non via un true/false)

L'idée est bonne. Toutefois, je pense qu'il peut être bénéfique de pas trop surchargée ta PR actuelle, à toi de voir 😉

À la limite, si l'on veut simplifier nos échanges, on peut demander à @Master13011 de mettre en place un Discord...

[Master13011]

Bonjour,

Je vais valider ta MR sans créer de release pour le moment,

Il faudra rebase cette branche : #18

Tu peux en cas travailler sur la branche de ProBreizh35:main