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

Automl save/load: add optional "protocol" argument for cloudpickle #821

Closed
gsheni opened this issue May 29, 2020 · 0 comments · Fixed by #1023
Closed

Automl save/load: add optional "protocol" argument for cloudpickle #821

gsheni opened this issue May 29, 2020 · 0 comments · Fixed by #1023
Assignees
@dsherry
Labels
enhancement An improvement to an existing feature. good first issue Issues which would be a good starting point for new hires.
Milestone
August 2020

Comments

@gsheni
Copy link
Contributor

gsheni commented May 29, 2020 •
edited

  • Since we use cloudpickle for saving a pipeline, we should allow the user to pass additional arguments, such as protocol

  • Same follows for loading
@gsheni gsheni added enhancement An improvement to an existing feature. good first issue Issues which would be a good starting point for new hires. labels May 29, 2020
@dsherry dsherry changed the title Save and Load Functions should have additional arguments Automl save/load: add optional "protocol" argument for cloudpickle Aug 5, 2020
@dsherry dsherry self-assigned this Aug 5, 2020
@dsherry dsherry added this to the August 2020 milestone Aug 5, 2020
@dsherry dsherry mentioned this issue Aug 5, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dsherry dsherry

Labels
enhancement An improvement to an existing feature. good first issue Issues which would be a good starting point for new hires.
Projects
None yet
Milestone
August 2020
Development

Successfully merging a pull request may close this issue.

Add component save/load; add protocol argument to all save/load alteryx/evalml
2 participants
@dsherry @gsheni