-
Notifications
You must be signed in to change notification settings - Fork 95
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
Summary is calculated wrong? #78
Comments
Hi, this is because the quantile is calculated using a linear interpolation when the quantile is between two values. But to be honest I implemented the function this way because I found a source on the php.net documentation (as mentionned here) and I didn't even know there were more than one way to calculate quantiles ! English wikipedia mentions 9 different ways to calculate it !!! So the questions are :
@LKaemmerling what do you think ? |
Hey, I guess we should follow the same way as the go client. @mp3000mp do you want to prepare the change :)? I guess we can do it as a bug fix. If not, I will try to come up with a fix in the next two weeks. |
I have the following samples
[1.33, 2.33, 2.33]
.Summary metric is returned like this:
Compared results with Go prometheus client and it return different result
Also tried some online percentile calculators:
So, I assume there is something wrong on PHP client
The text was updated successfully, but these errors were encountered: