Core: OpenWeatherMap shows daily morning weather

[metbril]

The problem

When using the integration with the freedaily mode, the Weather card shows the morning weather at 8:00, instead of the day weather. For example, the temperatures are always much too low compared to other weather integrations.

Dark Sky

OpenWeatherMap

OpenWeatherMap (attributes)

Environment

• Home Assistant Core release with the issue: 0.110.3
• Last working Home Assistant Core release (if known): ?
• Operating environment (Home Assistant/Supervised/Docker/venv): Home Assistant
• Integration causing this issue: OpenWeatherMap
• Link to integration documentation on our website: https://www.home-assistant.io/integrations/openweathermap/

Problem-relevant configuration.yaml

weather:
  - platform: openweathermap
    api_key: !secret openweathermap_api_key
    mode: freedaily

Traceback/Error logs

Additional information

The One Call API has daily forecasts for multiple times of the day:
https://openweathermap.org/api/one-call-api

I think the integration currently uses the daily.temp.morn but should use daily.temp.day. I haven't been able to find this in the integration code, so it might as well be a bug in the OWM library.

https://github.com/home-assistant/core/blob/fbe7b4ddfa813e300f99175d254ec509c8d65e33/homeassistant/components/openweathermap/weather.py#L5

[probot-home-assistant[bot]]

Hey there @fabaff, mind taking a look at this issue as its been labeled with a integration (openweathermap) you are listed as a codeowner for? Thanks!
_{^{(message by CodeOwnersMention)}}

[N1c093]

The one call api isn't implemented yet. They are working on it to integrate the one call api to the python wrapper pyowm https://github.com/csparpa/pyowm

After the python wrapper is updated they can go on and implement it into HA. There is already a feature request for it: https://community.home-assistant.io/t/convert-openweathermap-integration-to-use-one-call-api/186441

In the old API this behavour seems to be normal. It shows you the forecast for the next days, based on the current time.

[edenhaus]

The one call api isn't implemented yet. They are working on it to integrate the one call api to the python wrapper pyowm https://github.com/csparpa/pyowm

After the python wrapper is updated they can go on and implement it into HA. There is already a feature request for it: https://community.home-assistant.io/t/convert-openweathermap-integration-to-use-one-call-api/186441

In the old API this behavour seems to be normal. It shows you the forecast for the next days, based on the current time.

The OneCall Api is already implemented on the develop branch of pyown. The new version should be released soon.

[eloo]

The OneCall API is already released.
https://github.com/csparpa/pyowm/releases/tag/3.0.0

Sadly this is not part of the latest HA release.

[edenhaus]

The OneCall API is already released.
https://github.com/csparpa/pyowm/releases/tag/3.0.0

Sadly this is not part of the latest HA release.

There exists already plans to add it. I will add it after #34659 is merged.

[eloo]

@edenhaus sounds good :+1:

[nzapponi]

The OneCall API is already released.
https://github.com/csparpa/pyowm/releases/tag/3.0.0
Sadly this is not part of the latest HA release.

There exists already plans to add it. I will add it after #34659 is merged.

It seems like #34659 is now merged! Any ETA on when we can expect it to be available in HA?

[edenhaus]

The OneCall API is already released.
https://github.com/csparpa/pyowm/releases/tag/3.0.0
Sadly this is not part of the latest HA release.

There exists already plans to add it. I will add it after #34659 is merged.

It seems like #34659 is now merged! Any ETA on when we can expect it to be available in HA?

This month I have no time et all... Sorry I will look at it probably in October/November. If some one has time earlier please feel free to implement it.

[nzapponi]

FYI, done!