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

Trial display assumes the max GC comes from the sample with the max time #248

Open
mbauman opened this issue Aug 16, 2021 · 1 comment
Open
Labels

Comments

@mbauman
Copy link
Member

mbauman commented Aug 16, 2021

If I followed the breadcrumbs here right, the displayed max GC time comes from the maximum as computed here:

function Base.maximum(trial::Trial)
i = argmax(trial.times)
return TrialEstimate(trial, trial.times[i], trial.gctimes[i])
end

Which isn't really the maximum GC time. Since it's displayed independently, this can be surprising.

Cf. https://discourse.julialang.org/t/time-and-benchmark-give-different-results/66374/3

@vchuravy
Copy link
Member

In BenchmarkTools GC time is always printed as a ratio. That's why we do gcratio(maximum(trial)).

So yes the fact that we say the "maximum GC" is 0.0% is confusing

See discussion in #250 as well.

@gdalle gdalle added the bug label Jun 13, 2023
@gdalle gdalle added this to the v2.0 milestone Sep 18, 2023
@gdalle gdalle removed this from the v2.0 milestone Jan 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants