aqi-wtf icon indicating copy to clipboard operation
aqi-wtf copied to clipboard

Use 10 minute average instead of instantaneous data

Open skalnik opened this issue 3 years ago • 4 comments

Someone requested we use the 10 minute average instead of the latest value. I think that'd be good, but I believe we'll need to poll ThingSpeak instead of PA for that. Unsure how hard that'd be to implement, but I wanted to make note of it.

skalnik avatar Sep 10 '20 06:09 skalnik

Is there a -reason- they want the 10 minute averages?

One -possible- alternative to using the thingspeak API would be to compute our own rolling 5 or 10 minute average if we have the data and the sensor we're polling hasn't changed.

obra avatar Sep 10 '20 19:09 obra

(I don't believe that there's no reason to want 10 minute averages or that those averages aren't better. I am absolutely not skilled in that side of things. But I'm a big fan of documenting the "why" of changes.

obra avatar Sep 10 '20 19:09 obra

They said they preferred the 10 minute average to avoid sudden spikes/dips and smooth those out. I think that's fair, but also not a huge problem. I promised I'd document this but I'm not sure it's a huge priority.

skalnik avatar Sep 10 '20 21:09 skalnik

If we end up going down that path, it might be interesting to also look at averaging data from multiple sensors, ideally ones we're in the middle of. But that may be a really bad idea in non-densely-covered areas.

On Thu, Sep 10, 2020, at 2:44 PM, Mike Skalnik wrote:

They said they preferred the 10 minute average to avoid sudden spikes/dips and smooth those out. I think that's fair, but also not a huge problem. I promised I'd document this but I'm not sure it's a huge priority.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/skalnik/aqi-wtf/issues/50#issuecomment-690748531, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAALC2AGKJWEALYYUA67JU3SFFCFPANCNFSM4REMNFEQ.

obra avatar Sep 10 '20 23:09 obra