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

Changes in the 95th percentile aren't always visible in Glam when they are visible in Telemetry #1639

Closed
smaug---- opened this issue Oct 12, 2021 · 2 comments

Comments

@chutten
Copy link

chutten commented Oct 12, 2021

I have two thoughts on why this might occur (and thoughts about what GLAM could do):

  1. TMO's underpinning library telemetry.js v2 performs interpolation for its percentiles. This isn't really correct, but if we assume that the interpolation's being done on a continuous distribution that's just being approximated as a discrete distribution, then it's correct enough. And it's more sensitive to changes, this being just one of them.
    • GLAM could adopt interpolation (perhaps optionally) for numeric metric types that are likely discrete approximations of continuous phenomena
  2. If the population is, like it might be for this case, predominated by frequent users having a lovely time and infrequent users having absolutely awful image decode latency, GLAM might let that minority actually speak up sufficiently to be heard.
    • This would explain why GLAM's 95ths are so much higher (200k to 130k) than TMO's.

@edugfilho
Copy link
Collaborator

I believe this has been fixed with bucket interpolation (#2995) and non-normalized histograms (mozilla/bigquery-etl#3873)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants