-
Notifications
You must be signed in to change notification settings - Fork 509
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 llama download
support for multiple models with comma-separated list
#243
base: main
Are you sure you want to change the base?
Conversation
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.
Thanks! LG!
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.
Could you install pre-commit and lint / format the code?
elif args.model_id: | ||
model_ids = [args.model_id] | ||
else: | ||
parser.error("Please provide a model id or a list of model ids (--model-ids)") |
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.
nit: Please provide a model using --model-id or --model-ids
@@ -48,6 +48,11 @@ def setup_download_parser(parser: argparse.ArgumentParser) -> None: | |||
required=False, | |||
help="See `llama model list` or `llama model list --show-all` for the list of available models", |
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 just make this nargs='+'
so you can provide it multiple times like --model-id A --model-id B
?
Address issue #233 , allowing users to pass a comma-separated list of model IDs with the --model-ids argument and matching meta URLs via --meta-url.