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

Added BentoML framework to "Model Deployment and Orchestration Frameworks" #67

Merged
merged 2 commits into from Apr 6, 2020

Conversation

parano
Copy link
Contributor

@parano parano commented Apr 6, 2020

Hi - I'd like to add BentoML to the list of Model Deployment and Orchestration Frameworks. It's a framework agnostic serving system, that does dependency management, model management, and high-performance API serving.

I think it might make more sense to separate model serving frameworks such as BentoML, Clipper, MXNet Model Server, TF-serving and deployment orchestration frameworks such as Seldon, KFServing, Redis-AI, etc since people usually pick one serving framework and another deployment framework independently. Happy to make that contribution as well if I could get consensus from the community.

Cheers!

@axsaucedo
Copy link
Collaborator

Looks great @parano ! Can you append it instead of prepending? I've been meaning to convert all to alphabetical order but would be quite a feat so haven't gotten around to it, but for the meantime let's append it to the model orchestration

@parano
Copy link
Contributor Author

parano commented Apr 6, 2020

Thanks, @axsaucedo! totally thought that list was already in alphabetical order ;) updated!

@axsaucedo
Copy link
Collaborator

Thank you! I wish... soon it will be hopefully 😄

@axsaucedo axsaucedo merged commit 29b8c6b into EthicalML:master Apr 6, 2020
@parano
Copy link
Contributor Author

parano commented Apr 6, 2020

Thanks @axsaucedo!

VIM has a built-in command :sort to sort selected lines alphabetically, happy to help out on that if needed!

@axsaucedo
Copy link
Collaborator

Oh that's very useful, does taht work on selection as well? Only inconvenience I see is that the lines are prefixed with "`* [" but I guess that could be removed to run that command

If you'd be able to do that, it would be awesome!

@parano
Copy link
Contributor Author

parano commented Apr 6, 2020

@axsaucedo yes it works on selections, and I think it works if all lines are prefixed with * [, I will give it a try!

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.

None yet

2 participants