-
Notifications
You must be signed in to change notification settings - Fork 145
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
Create a Docker image for Morpheus models #1804
Create a Docker image for Morpheus models #1804
Conversation
… validation scripts but without the models
…v-morpheus#1764)" This reverts commit ba23b31.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
followed the new readme instructions, and tested all the models in the new model container. lgtm
This works for me
Tag it the same as the Morpheus version. While its possible to change the Triton version used, we do tie our releases to a specific Triton version
Lets go with the "dont deploy" option. If it takes seconds, the barrier to entry is low. These containers are going to be large due to the models being included. Best to avoid pushing unnecessary alpha tags |
/merge |
Description
models/docker/Dockerfile
along with a helper scriptmodels/docker/build_container.sh
examples/ransomware_detection/models
intomodels/ransomware-models
Closes #552
By Submitting this PR I confirm: