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

llm models -q should support multiple -q options, should work for embedding models too #748

Closed
simonw opened this issue Feb 13, 2025 · 2 comments
Labels
enhancement New feature or request

Comments

@simonw
Copy link
Owner

simonw commented Feb 13, 2025

This should work:

llm models -q hf -q zeta

Equivalent of a search for anything substring matching both hf and zeta.

Plus llm embed-models should support the same option.

@simonw simonw added the enhancement New feature or request label Feb 13, 2025
@simonw
Copy link
Owner Author

simonw commented Feb 13, 2025

I fed llm/cli.py into o3-mini with this prompt:

Output a fully copy of this file with the following changes:

  1. The -q/--query option on models_list should me a multi - and when multiple strings are provided it returns models matching all of them
  2. Add that option to the embed_models_list command as well
  3. The aliases_set command should make the model_id optional but only if you provide one or more -q/--query options - if you do it runs that search and assigns the alias for the first matching model, then outputs a message with click.echo(..., err=True) telling you what it did

Result: https://gist.github.com/simonw/33108b29621ad67dbc6b0395b4b21039

simonw added a commit that referenced this issue Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant