pkissling / clock-weather-card

A Home Assistant Card indicating today's date/time, along with an iOS inspired weather forecast for the next days with animated icons
Other
304 stars 65 forks source link

Higher night temp value causes display issues #324

Closed patriiiiiiiiiick closed 5 months ago

patriiiiiiiiiick commented 5 months ago

Checklist:

Release with the issue: 2.2.2 Last working release (if known):

Browser and Operating System: Android companion app

Description of problem: 298386571-6900ab32-3620-44e2-8597-a0deb7e6ec09

Same data have been used for both cards above. Wednesday is missing for clock weather card. This is probably due to the night temperature being higher than the day one.

More details and incorrect behaviour are described in https://github.com/jdejaegh/irm-kmi-ha/issues/8 but I suspect all is linked.

Javascript errors shown in the web inspector (if applicable):

Additional information:

pkissling commented 5 months ago

hi @patriiiiiiiiiick

according to the home assistant documentation attributes temperature and templow define the highest and lowest temperature of the given day respectively. Screenshot 2024-01-21 at 18 59 29

thus it has nothing to do with day / night temperatures. this looks like a data issue to me, since in your screenshot the lower temperature of the day is higher than the highest temperature, which breaks the calculation logic in clock-weather-card

patriiiiiiiiiick commented 5 months ago

Thank you for the feedback. And congratulations to both @pkissling and @jdejaegh for the collaborative attitude.