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

Add a bound to the inference tips cache (and fix regressions from #2158) #2181

Merged
merged 5 commits into from
May 15, 2023
Merged

Add a bound to the inference tips cache (and fix regressions from #2158) #2181

merged 5 commits into from
May 15, 2023

Commits on May 15, 2023

  1. Remove cache entries in all exit paths 

    Follow-up to 0740a0d.
    @jacobtylerwalls
    jacobtylerwalls committed May 15, 2023
    Configuration menu
    Copy the full SHA
    6bdd92d View commit details
    Browse the repository at this point in the history

  2. Add InferenceContext.is_empty()

    @jacobtylerwalls
    jacobtylerwalls committed May 15, 2023
    Configuration menu
    Copy the full SHA
    c807c03 View commit details
    Browse the repository at this point in the history

  3. Add a bound to the inference tips cache 

    Small bounds still yield about equal hits and misses.

Further work could determine if storing only the last result
is optimal.
    @jacobtylerwalls
    jacobtylerwalls committed May 15, 2023
    Configuration menu
    Copy the full SHA
    4757af2 View commit details
    Browse the repository at this point in the history

  4. Add disable

    @jacobtylerwalls
    jacobtylerwalls committed May 15, 2023
    Configuration menu
    Copy the full SHA
    5922183 View commit details
    Browse the repository at this point in the history

  5. Skip recursion test on PyPy 

    Reapplied from c1e4c95.
    @jacobtylerwalls
    jacobtylerwalls committed May 15, 2023
    Configuration menu
    Copy the full SHA
    5a33300 View commit details
    Browse the repository at this point in the history