Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Batch gauge data to create more realistic gauge projections for a given polling interval #6

Open
mplachter opened this issue Dec 1, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@mplachter
Copy link
Contributor

As a user, I would like to be able to Batch incoming Gauge data and then on a interval compute the sum and set it as the correct aggregate for that given resolution period.

@mplachter mplachter added the enhancement New feature or request label Dec 1, 2022
@sontek
Copy link
Contributor

sontek commented Jan 5, 2023

@mplachter Did you make this change? I know you were working on some gauge work

@mplachter
Copy link
Contributor Author

I did not get around to this one yet. Right now all gauge data is still just aggregated together

@mplachter
Copy link
Contributor Author

mplachter commented Jan 6, 2023

It looks like he took an average gauge value approach 🤔

I think that's half of the equation looks like they always keep values and keep averages going... I think over a long period of time this will not be the fairest representation of the gauge data for example.

Last AVG was 10 new data coming in is 100 + 100 + 100 it will say the gauge average 88

(10 + 100) / 2 = 55 
(55 + 100) / 2 = 77
(77 + 100) / 2 = 88

But the real average during that gauge interval should have been 100.

We still do need the code to average them out, I think we definitely should look at the performance of the fingerprinting implemented in the above implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants