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

TST Add trust check to persistence performance test #351

Commits on Apr 24, 2023

  1. Add trust check to persistence performance test

    Description
    
    Our script to check that skops persistence is not too slow compared to
    pickle used trusted=True so far. However, this is not the recommended
    way of using skops -- ideally the check for trusted types should be
    performed, which theoretically could slow down persistence. This PR adds
    the check for trusted types to the script.
    
    Results
    
    Testing locally, there is only a small difference. For instance, for the
    slowest couple of tests, the added time was only 0.05 sec or less. On
    average across all estimators, the difference was 0.004 sec. or 30%
    slower. This relatively small increase means that the check for trusted
    types is fast compared to constructing the objects.
    BenjaminBossan committed Apr 24, 2023
    Configuration menu
    Copy the full SHA
    fa085b5 View commit details
    Browse the repository at this point in the history