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.

Sign up for GitHub

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

Jump to bottom

Look into accuracy of movingstd() in Timelion #9792

Closed
tbragin opened this issue Jan 9, 2017 · 5 comments · Fixed by #15911
Closed

Look into accuracy of movingstd() in Timelion #9792

tbragin opened this issue Jan 9, 2017 · 5 comments · Fixed by #15911
Assignees
@thomasneirynck
Labels
bug Fixes for quality problems that affect the customer experience Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available)

Comments

@tbragin
Copy link
Contributor

tbragin commented Jan 9, 2017 •
edited
Loading

There have been reports of discrepancies in the movingstd() function in Timelion, we should look into that.

More details here: elastic/timelion#177
@tbragin tbragin added Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available) bug Fixes for quality problems that affect the customer experience P2 labels Jan 9, 2017
@thomasneirynck thomasneirynck self-assigned this Jan 9, 2017
@tbragin
Copy link
Contributor Author

tbragin commented Jan 9, 2017

@polyfractal per our chat, feel free to work with @thomasneirynck on this if you have any insights. would be good to get your input.

@thomasneirynck
Copy link
Contributor

similar issue logged here: https://discuss.elastic.co/t/timelions-movingstd-vs-kibanas-standard-deviation/50373

@tbragin tbragin added P3 and removed P2 labels Feb 28, 2017
@tbragin
Copy link
Contributor Author

tbragin commented Mar 21, 2017

@thomasneirynck @polyfractal @alexfrancoeur since we don't have time to look into it, should we mark this function as "experimental" for now, like we do with some other functions in Timelion?

@polyfractal
Copy link
Contributor

Should we maybe just yank it for now? At this point it's probably just classified as "buggy" rather than experimental, since it's giving potentially incorrect values.

I'm unfortunately still underwater with other work, too much to do at the moment on other projects and not enough time to spin up a kibana dev environment again :/

@thomasneirynck thomasneirynck mentioned this issue Apr 5, 2017
Closed
@mbertani mbertani mentioned this issue Apr 6, 2017
Closed
@epixa epixa removed the P3 label Apr 25, 2017
@nreese nreese mentioned this issue Jan 9, 2018
Merged
@thomasneirynck
Copy link
Contributor

This has been fixed with #15911

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thomasneirynck thomasneirynck

Labels
bug Fixes for quality problems that affect the customer experience Feature:Timelion Timelion app and visualization Feature:Visualizations Generic visualization features (in case no more specific feature label is available)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[timelion] fix movingstd function nreese/kibana
4 participants
@epixa @tbragin @polyfractal @thomasneirynck