-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
Pickle works with the old API not the new one #4985
Comments
Could you post a minimal standalone python example on synthetic trivial data instead of the notebook? Thx |
@karlnapf I have updated the link, now it's a python file using Shogun's data however you're going to have to edit the path to point to your shogun dir |
thanks. that does it. |
Could you share Traceback if there is one please? |
@gf712 I have updated the gist |
I think the offender is this hackery in
Simple solution would be to replace it with the |
Which templated objects are you referring to? The template type is hidden from interfaces no? |
Nevermind for now, I thought this could become a problem, but not sure anymore now, as the serialization and the new api have changed the picture. I guess someone will see when trying to fix this |
Whenever Pickle loads a saved Shogun model, it expects the model to be implemented using the old API this can be demonstrated here
I guess this is related to #4463
The text was updated successfully, but these errors were encountered: