-
-
Notifications
You must be signed in to change notification settings - Fork 28.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Meteo france integration is not using the new observation endpoint #105900
Comments
Hey there @hacf-fr, @oncleben31, @Quentame, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) meteo_france documentation |
There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. |
Ths issue is not resolved |
There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. |
The problem
Hello,
The problem I face is a very noisy current temperature returned by the Meteo France integration:
After some digging I saw that the underlying
meteofrance_api
package had introduced a newget_observation
method to solve this issue: hacf-fr/meteofrance-api#520.But it looks like the HA integration never took this new feature into account, I don't see any call to this method in the code: https://github.com/home-assistant/core/blob/master/homeassistant/components/meteo_france/__init__.py#L74.
Is it intended? If not I can submit a PR if you wish
Thanks a lot
What version of Home Assistant Core has the issue?
core-2023.11.2
What was the last working version of Home Assistant Core?
No response
What type of installation are you running?
Home Assistant Container
Integration causing the issue
meteo_france
Link to integration documentation on our website
https://www.home-assistant.io/integrations/meteo_france/
Diagnostics information
No response
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response
The text was updated successfully, but these errors were encountered: