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

Save class definition (instead of instance) #91

Closed
adriangonz opened this issue Apr 8, 2021 — with Board Genius Sync · 0 comments · Fixed by #105
Closed

Save class definition (instead of instance) #91

adriangonz opened this issue Apr 8, 2021 — with Board Genius Sync · 0 comments · Fixed by #105

Comments

Copy link

For decorated classes, it's interesting in some cases to store the class definition instead of the class instance. This means that instead of doing,

myc = MyClass()
myc.save()

One could do

MyClass.save()

This in turn would require the MLServer runtime to call MyClass() when it loads the artifact from the remote URI.

@adriangonz adriangonz self-assigned this Apr 8, 2021
@adriangonz adriangonz added this to To do in Early-Access Release via automation Apr 8, 2021
@adriangonz adriangonz added this to To do in MLOps Sprint 5 (ending 15/04/2021) via automation Apr 8, 2021
@axsaucedo axsaucedo added this to To do in MLOps Sprint 6 (29/04/2021) via automation Apr 15, 2021
@ukclivecox ukclivecox moved this from To do to In progress in MLOps Sprint 6 (29/04/2021) Apr 22, 2021
@ukclivecox ukclivecox moved this from To do to In progress in Early-Access Release Apr 22, 2021
Early-Access Release automation moved this from In progress to Done Apr 28, 2021
MLOps Sprint 6 (29/04/2021) automation moved this from In progress to Done Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

1 participant