-
Notifications
You must be signed in to change notification settings - Fork 24
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
Error bar isn't centralized #19
Comments
Hey @danielmenezesbr do you mean vertically centralized ? The error bar shows goes from min to max score (see #11), so thats why it's not symmetric! |
Thanks @jzillmann Perhaps to be clearer it should be documented that the error bar uses min/max score instead of the confidence interval. |
I agree this isn't the expected or (in my case) desired behavior. I expect to see the 99% confidence interval in decimal format (not integers). I'll give you a concrete example. I just benchmarked two methods:
As you can see, the confidence intervals do not overlap. The second method is faster than the first 99% of the time. Yet when I view this data in the jmh-visualizer the error bars overlap and it's not clear that this is the case. |
We used the confidence interval for a long time and i was hesitating to change it - see #11. However i myself and others run into a lot of cases where it wasn't that meaningful. I guess it depends on the benchmarks and its iteration time what is more useful. If you execute a benchmark only for very few times (like 10 iterations and 2 forks), the 99th percentile / confidence interval has normally a big skew... Maybe it makes sense to make this configurable ? How are you guys using jmh-visualizer ? The web version ? Gradle ? Or Jenkins ? |
Web version in my case. |
It seems a good idea to make this configurable. Web version too. |
The text was updated successfully, but these errors were encountered: