[Bug] Generic Weather Day index mismatch with Breezy weather?

[twicenothere]

I'm using BreezyWeather as the Target for the Generic Weather plugin, and it looks like the day name vs weather is mismatched. I've been watching it for a couple days now and the behavior appears consistent.

In the image below you can see the plugin, and underneath the BreezyWeather widget.

The Plugin target is showing the days of the week starting with yesterday (today is Friday, so it's showing yesterday, today, Saturday and Sunday). However, for the 4 day temperature shown, it's pulling data for tomorrow (Saturday), Sunday, Monday and Tuesday. The choice of weather to pull is reasonable to me since the current day is shown on the right.

Note there is a current mismatch is temperature between Saturday/Monday that I think is just an update interval issue. Normally these match.

Maybe there's a configuration or use error? Also happy to take a few more screenshots or something.

This is the only mismatch I've noticed too. Otherwise it works great!

[pacjo]

Hi, thanks for a detailed issue. Sadly I don't see the same mismatch on my end.

Target refreshes whenever Breezy Weather does, so I'm thinking this might be some update related problem.

In no particular order:

• do you have automatic (background) updates enabled in Breezy? If so what's the refresh period?
• does manually forcing refresh in Breezy fix the mismatch?
• are there any moments when there's no mismatch between GenericWeather target and Breezy widget?

[twicenothere]

Ok, I'm just misinterpreting something then since I see the same offset on your screenshot as well.

I'm reading the value displayed as the high of the day. I annotated the image below to show why I was thinking there an indexing problem.

For completeness:

• automatic updates in breezy are enabled. 1.5hr period
• manual refresh updates the numbers, but the behavior in observing doesn't change
• no observed change, the behavior is consistent

[pacjo]

Oh, ok. Didn't see that. Interesting. I'll have to take a look at the indexing code to see where do the days come from.

this:

SmartspacerPlugins/app/src/genericweather/src/java/targets/WeatherConditionTarget.kt

Line 66 in c20553b

second = Time(provideContext(), Instant.now().truncatedTo(ChronoUnit.DAYS).plus(index.toLong(), ChronoUnit.DAYS).getLong(ChronoField.INSTANT_SECONDS)).getEventDate( "EEE")

I really don't like this code and looking at it right now it looks like it might be a broken mess. Unfortunately as I mentioned in another issue I don't have time nor access to my setup right now, so there's not much I can do about it.

[twicenothere]

Totally understand. Appreciate you even taking a look!