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

Add custom loaders for function notation #75

Merged
merged 25 commits into from
Apr 19, 2021

Conversation

adriangonz
Copy link

@adriangonz adriangonz commented Mar 31, 2021

Fixes #17

Changelog

  • Move most of Model to BaseModel
  • Remove cls pointer and simplify how custom functions are bound to the class
  • Add loadmethod decorator when using function notation
  • Add context field to share object instances between different methods (useful when using the function notation)
  • Improve how pipelines / models get initialised in MLServer
  • Add example for custom model with custom artifacts

TODO

@adriangonz adriangonz changed the title Add custom loaders for function notation WIP : Add custom loaders for function notation Mar 31, 2021
@adriangonz adriangonz force-pushed the load-method branch 2 times, most recently from 1b19bde to 78d3f15 Compare April 6, 2021 16:21
@adriangonz adriangonz changed the title WIP : Add custom loaders for function notation Add custom loaders for function notation Apr 8, 2021
@ukclivecox ukclivecox merged commit f9aa6de into SeldonIO:master Apr 19, 2021
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 this pull request may close these issues.

Handler additional artifacts and custom load methods
2 participants