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

Increase the benchmark warmup time to avoid warmup artifact #355

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Increase the benchmark warmup time to avoid warmup artifact #355

wants to merge 1 commit into from

Commits on Sep 25, 2024

  1. Increase the benchmark warmup time to avoid warmup artifact 

    This commit increases the warmup time used in our benchmark so that two
benchmarked gems with similar code (e.g. local `MemoWise` vs. the
version pulled from GitHub) will not have their benchmark results depend
on whichever happens to run second (which the Ruby interpreter has had
more time to optimize). This issue was not seen in CI but was reported
by one user running benchmarks locally.
    @JacobEvelyn
    JacobEvelyn committed Sep 25, 2024
    Configuration menu
    Copy the full SHA
    a43489d View commit details
    Browse the repository at this point in the history