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

Usability issue with skops #400

Closed
tuscland opened this issue Sep 24, 2024 · 0 comments · Fixed by #420
Closed

Usability issue with skops #400

tuscland opened this issue Sep 24, 2024 · 0 comments · Fixed by #420
Assignees

Comments

@tuscland
Copy link
Member

Even with bona fide scikit-learn estimators, skops might fail to recognize some properties as trusted, and therefore fail to load the saved object. We should work around this limitation by temporarily disabling skops's security feature until we are able to put more efforts in supporting it.

At save time, query skops to get the list of warnings. Save the list on the side, then at load time, provide the skops with the "trusted" list.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants