pirateweather icon indicating copy to clipboard operation
pirateweather copied to clipboard

Daily high/low temperature calculation issue

Open cloneofghosts opened this issue 7 months ago • 4 comments

Describe the bug

I've noticed that the daily high and low temperatures do not take into account 6pm for the daily high and 6am for the daily low. If the daily high temperature is at 6pm then that temperature isn't shown as the daily high. Same goes for the nightly low if its at 6am then it also isn't used as the daily low temperature.

Expected behavior

Daily high should include 6pm and daily low should include 6am when calculating the high and low temperatures

Actual behavior

Daily high does not include 6pm and daily low does not include 6am when calculating the high and low temperatures

API Endpoint

Production

Location

Ottawa, Ontario

Other details

I had posted this comment in #239 a few weeks ago but it looks like it was missed so posting here for better visibility.

Troubleshooting steps

  • [X] I have searched this repository and Home Assistant Repository to see if the issue has already been reported.
  • [X] I have read through the API documentation before opening this issue.
  • [X] I have written an informative title.

cloneofghosts avatar Jul 16 '24 14:07 cloneofghosts