-
Notifications
You must be signed in to change notification settings - Fork 627
feat: support cog init with pipeline template #2437
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
Conversation
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
@@ -0,0 +1,27 @@ | |||
# This is a normal Python requirements.txt file. |
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.
Can we remove this file? We don't support additional python packages in pipelines so it might lead to user confusion
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.
in the web editor it has these contents:
# IMPORTANT: The requirements.txt is only for local development,
# it will be ignored in production.
replicate>=1.1.0b1,<2.0.0a1
What is considered 'local' ?
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.
Since we don't support requirements.txt (aka custom dependencies) for pipelines running on Replicate, and we expect pretty much everyone to be developing pipelines so that they can eventually run them on Replicate (vs wanting to run them purely locally), we should just build the init under that assumption.
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
Signed-off-by: Mark Phelps <209477+markphelps@users.noreply.github.com>
…replicate/cog into feat/add-init-pipelines-support * 'feat/add-init-pipelines-support' of https://github.com/replicate/cog: Move to CDN for monobase matrix.json
Fixes: DP-85
Adds the
--x-pipeline
flag tocog init
to create a pipeline template for cogExample
Notes/Questions
--x-pipeline
flag hidden (from cli usage) as this matchescog push
where-x-pipeline
is also a hidden flag--x-pipeline
--use-replicate-token
flags?