Page 1 of 1

Display Units

PostPosted: Tue Mar 13, 2018 12:00 pm
by Monstergerm
I think these are items that got lost/modified during the v1 to v7 transition:


Rain fall amount units specified under device setup are omitted for all Rain Amount - Day 1 (Day 2 etc.) variables.

Re: Display Units

PostPosted: Wed Mar 14, 2018 12:28 pm
by DaveL17
Sorry for the delay in responding—I’m out of town for work.

I appreciate the head’s up on this. It’s obviously not an intentional change and I’ll definitely look into it. I’ve been in the code for this plugin, so the timing is perfect. I suspect it’s a bug that came up in the transition to the way I ship state changes to the Indigo server under API 2.0.


Sent from my iPhone using Tapatalk

Re: Display Units

PostPosted: Wed Mar 14, 2018 1:15 pm
by Monstergerm
No rush at all, this is just minor thing.
I think it has to do with rainUnits versus rainAmountUnits. The unit display on the weather device and hourly device is fine. But on the 10-day device no units are shown.

Re: Display Units

PostPosted: Wed Mar 14, 2018 1:23 pm
by DaveL17
I’m going to take a deeper dive on this.

At some point, I moved this preference from the plugin config (to affect all plugin devices) to device specific settings for folks that might want to do different units for different devices. In looking at the code now, I think it could be simplified (and fixed in this case)


Sent from my iPhone using Tapatalk

Re: Display Units

PostPosted: Wed Mar 14, 2018 7:29 pm
by DaveL17
Monstergerm wrote:
I think it has to do with rainUnits versus rainAmountUnits.

That is precisely right. I had an overly aggressive try block that was always returning no units for the 10-day device. Will be fixed in the upcoming release.

Thanks for letting me know.