search

hacf-fr / lovelace-meteofrance-weather-card

⛅ Carte Lovelace dédiée à l'intégration Météo-France pour Home Assistant.
MIT License
54 stars 16 forks source link

Add windspeed to forecast #102

Closed vingerha closed 1 year ago

vingerha commented 1 year ago

Adds wind speed to the hourly forecast. At present this is misisng from the implementation Below an example of the enhancement, note that the post-sunset icons are fixed in a separate PR image

dx44 commented 1 year ago

La bonne pratique est aussi de commenter les PRs avec une description de ce qui est amélioré ou corrigé. Pour les ajouts dans l'interface utilisateur, présenter une capture de la nouvelle carte avec la mise en évidence des modifications, ce qui permet de valider plus rapidement et peut éviter une perte de temps lors du test avant release.

vingerha commented 1 year ago

Clear et fait

WarC0zes commented 1 year ago

Bonjour, j'ai tester ce PR et je vois que la vitesse du vent est différente de ce que je vois dans les attributs de mon entité météo. vitesse vent vitesse vent 2

En regardant le code je vois ${Math.round(daily.wind_speed * 3.6)} ${this.getUnit("speed")} Pour mon information, pourquoi c'est multiplier par 3.6 ?

vingerha commented 1 year ago

Bien vû et je me rapelle maintenant pourquoi je l'avais fait il y a plus d'un an ...mon entity etait en m/s et pas km/h. J'ai fait des install fraiche de meteo-france et le tout montre comme km/h (comme chez toi) donc je corrige @dx44 voir nouveaux PR

vingerha commented 1 year ago

@WarC0zes En attendant le review, si tu veux tester la version combiné (icones + windspeed), c'est release 0.9 ... dans mon rep. https://github.com/vingerha/lovelace-meteofrance-weather-card

WarC0zes commented 1 year ago

@WarC0zes En attendant le review, si tu veux tester la version combiné (icones + windspeed), c'est release 0.9 ... dans mon rep. https://github.com/vingerha/lovelace-meteofrance-weather-card

Tester et fonctionnel pour le vent et les icones de nuit.

Edit: un soucis pour les icones de nuit sur une entité météo par jours. Par heure ca l'air bon. lune journée

vingerha commented 1 year ago

Merci, je vais revoir

vingerha commented 1 year ago

@WarC0zes resolu (de mon avis) release 91_combined

WarC0zes commented 1 year ago

@WarC0zes resolu (de mon avis) release 91_combined

Oui, c'est bon de mon coté aussi avec la 91_combined.

dx44 commented 1 year ago

@vingerha ou @WarC0zes , merci de présenter une nouvelle PR qui corrige celle-ci. Je l'intégrerai dans la pre-release suivante.

vingerha commented 1 year ago

Top...merci !