Carte Météo France - Prévision de pluie dans l'heure pour ceux qui n'ont pas de sensor next_rain

Flapoly · Octobre 9, 2024, 6:39

Bonsoir,

il ne faut pas mettre de majuscule à « Template ». ^^

c’est corrigé. Merci

Argonaute · Octobre 9, 2024, 10:04

Salut,
Merci pour le tuto sur github. Mais il va quand même falloir que l’on fasse évoluer cette carte qui est sur notre github.
Au pire, je m’y collerai et ferai moi la modif. Je vais essayer de creuser le sujet.

Flapoly · Octobre 10, 2024, 6:32

Bonsoir,

Pour la carte, il faut déjà regarder ce qui a été fait sur la version de dev: https ://github.com/hacf-fr/lovelace-meteofrance-weather-card/tree/dev plusieurs améliorations sont présentes. Effectivement d’autres seraient les bienvenues.
Pour le sensor _next_rain la remediation est à faire dans HA.core. Il y a déjà une PR #127665 qui est en attente d’approbation.

Giga77 · Octobre 12, 2024, 9:54

Salut

J’ai fait une PR. Si personne ne la valide, je fais le merge dans la semaine et je crée une nouvelle release.

Giga77 · Octobre 21, 2024, 7:38

La carte Météo France n’est plus en version béta :

WarC0zes · Octobre 22, 2024, 7:45

Salut @Flapoly ,
merci pour ce PR, qui é été merge et release:

Horizontal scroll for daily & hourly by @Flapoly

C’est pratique pour avoir plus de jour ou heure sur la même carte

Argonaute · Octobre 24, 2024, 9:28

Excellente nouvelle !!
Par contre, on est d’accord que cela ne règle pas le pb de ne pas avoir l’entité next_rain (le sujet initial du post). L’entité est créée ou pas par l’intégration MeteoFrance.

WarC0zes · Octobre 25, 2024, 2:08

Oui, le problème n’est toujours pas corrigé. C’est du côté de l’intégration météo France qu’il faut le corrigé, pas de la carte.

github.com/home-assistant/core

Remove obsolete check for rain_product_available in Meteo France

home-assistant:dev ← Flapoly:meteofrance-next_rain

opened 06:06PM - 05 Oct 24 UTC

Flapoly

+11 -17

## Proposed change  Meteo France does now allow 1 hour rain forecast to all french cities. Test based on `rain_product_available` is no more accurate, and in consequence removed ## Type of change  - [ ] Dependency upgrade - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New integration (thank you!) - [X] New feature (which adds functionality to an existing integration) - [ ] Deprecation (breaking change to happen in the future) - [ ] Breaking change (fix/feature causing existing functionality to break) - [ ] Code quality improvements to existing code or addition of tests ## Additional information  - This PR fixes or closes issue: fixes #90397 - This PR is related to issue: - Link to documentation pull request: ## Checklist  - [X] The code change is tested and works locally. - [X] Local tests pass. **Your PR cannot be merged unless tests pass** - [ ] There is no commented out code in this PR. - [X] I have followed the [development checklist][dev-checklist] - [ ] I have followed the [perfect PR recommendations][perfect-pr] - [ ] The code has been formatted using Ruff (`ruff format homeassistant tests`) - [ ] Tests have been added to verify that the new code works. If user exposed functionality or configuration variables are added/changed: - [ ] Documentation added/updated for [www.home-assistant.io][docs-repository] If the code communicates with devices, web services, or third-party tools: - [ ] The [manifest file][manifest-docs] has all fields filled out correctly. Updated and included derived files by running: `python3 -m script.hassfest`. - [ ] New or updated dependencies have been added to `requirements_all.txt`. Updated by running `python3 -m script.gen_requirements_all`. - [ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.  To help with the load of incoming pull requests: - [ ] I have reviewed two other [open pull requests][prs] in this repository. [prs]: https://github.com/home-assistant/core/pulls?q=is%3Aopen+is%3Apr+-author%3A%40me+-draft%3Atrue+-label%3Awaiting-for-upstream+sort%3Acreated-desc+review%3Anone+-status%3Afailure  [dev-checklist]: https://developers.home-assistant.io/docs/development_checklist/ [manifest-docs]: https://developers.home-assistant.io/docs/creating_integration_manifest/ [quality-scale]: https://developers.home-assistant.io/docs/integration_quality_scale_index/ [docs-repository]: https://github.com/home-assistant/home-assistant.io [perfect-pr]: https://developers.home-assistant.io/docs/review-process/#creating-the-perfect-pr

github.com/home-assistant/core

Remove check for obsolete "rain_product_available" in meteo_france

home-assistant:dev ← Passific:patch-1

opened 07:04PM - 16 Oct 24 UTC

Passific

+9 -18

## Proposed change  Remove obsolete check for rain_product_available to allow 1h rain forcast for all. ## Type of change  - [ ] Dependency upgrade - [x] Bugfix (non-breaking change which fixes an issue) - [ ] New integration (thank you!) - [ ] New feature (which adds functionality to an existing integration) - [ ] Deprecation (breaking change to happen in the future) - [ ] Breaking change (fix/feature causing existing functionality to break) - [ ] Code quality improvements to existing code or addition of tests ## Additional information  - This PR fixes or closes issue: fixes #125693 #90397 - This PR is related to issue: - Link to documentation pull request: https://github.com/home-assistant/home-assistant.io/pull/35257 ## Checklist  - [x] The code change is tested and works locally. - [x] Local tests pass. **Your PR cannot be merged unless tests pass** - [x] There is no commented out code in this PR. - [x] I have followed the [development checklist][dev-checklist] - [x] I have followed the [perfect PR recommendations][perfect-pr] - [x] The code has been formatted using Ruff (`ruff format homeassistant tests`) - [x] Tests have been added to verify that the new code works. If user exposed functionality or configuration variables are added/changed: - [x] Documentation added/updated for [www.home-assistant.io][docs-repository] If the code communicates with devices, web services, or third-party tools: - [ ] The [manifest file][manifest-docs] has all fields filled out correctly. Updated and included derived files by running: `python3 -m script.hassfest`. - [ ] New or updated dependencies have been added to `requirements_all.txt`. Updated by running `python3 -m script.gen_requirements_all`. - [ ] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.  To help with the load of incoming pull requests: - [ ] I have reviewed two other [open pull requests][prs] in this repository. [prs]: https://github.com/home-assistant/core/pulls?q=is%3Aopen+is%3Apr+-author%3A%40me+-draft%3Atrue+-label%3Awaiting-for-upstream+sort%3Acreated-desc+review%3Anone+-status%3Afailure  [dev-checklist]: https://developers.home-assistant.io/docs/development_checklist/ [manifest-docs]: https://developers.home-assistant.io/docs/creating_integration_manifest/ [quality-scale]: https://developers.home-assistant.io/docs/integration_quality_scale_index/ [docs-repository]: https://github.com/home-assistant/home-assistant.io [perfect-pr]: https://developers.home-assistant.io/docs/review-process/#creating-the-perfect-pr

EDIT:
le deuxième PR vient d’être merge

WarC0zes · Novembre 1, 2024, 3:49

Bonjour,
bonne nouvelle pour le core 2024.11.0:

Remove check for obsolete “rain_product_available” in meteo_france (@Passific - #128533) ([meteo_france docs])

enfin, tout le monde va avoir une entité next rain

WarC0zes · Novembre 7, 2024, 4:28

Bonjour,
c’est OK, avec le core 2024.11.0, l’entité next rain est créé pour toutes les villes maintenant. Plus besoin de passé par l’api météo France et un template.