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

[UnitTest] Use pytest's scope='session' for tvm.testing.parameter #16930

Merged
merged 1 commit into from
May 13, 2024
Merged

[UnitTest] Use pytest's scope='session' for tvm.testing.parameter #16930

merged 1 commit into from
May 13, 2024

Commits on May 9, 2024

  1. [UnitTest] Use pytest's scope='session' for tvm.testing.parameter 

    Prior to this commit, the `tvm.testing.parameter` utility defined a
fixture with the default `scope="function"`.  However, this prevents
use of these parameters as arguments for other fixtures that are
themselves cached using pytest.  Since these are parameters, not large
values that would be expensive to compute, there is no downside to
caching them at the pytest level.

This commit updates the scope of fixtures generated using
`tvm.testing.parameter` to use `scope="session"` instead of the
default `scope="function"`.
    @Lunderberg
    Lunderberg committed May 9, 2024
    Configuration menu
    Copy the full SHA
    46a0483 View commit details
    Browse the repository at this point in the history