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 route to install huggingface models from model marketplace #6515

Merged
merged 11 commits into from
Jun 17, 2024

Conversation

chainchompa
Copy link
Contributor

Summary

added route to install huggingface models from model marketplace

Related Issues / Discussions

QA Instructions

test by going to http://localhost:5173/api/v2/models/install/huggingface?source=${hfRepo}

Merge Plan

Checklist

  • The PR has a short but descriptive title, suitable for a changelog
  • Tests added / updated (if applicable)
  • Documentation added / updated (if applicable)

@github-actions github-actions bot added api python PRs that change python files frontend PRs that change frontend files labels Jun 14, 2024
chainchompa and others added 8 commits June 14, 2024 17:15
Previously, we used `model_install_download_progress` for both download starting and progressing. When handling this event, we don't know which actual thing it represents.

Add `model_install_download_started` event to explicitly represent a model download started event.
When a model install is initiated from outside the client, we now trigger the model manager tab's model install list to update.

- Handle new `model_install_download_started` event
- Handle `model_install_download_complete` event (this event is not new but was never handled)
- Update optimistic updates/cache invalidation logic to efficiently update the model install list
@github-actions github-actions bot added the services PRs that change app services label Jun 17, 2024
@psychedelicious
Copy link
Collaborator

I've updated a few things:

  • Add a new model_install_download_started event to reduce ambiguity with model install events. See commits for details on why.
  • Update model install socket event handling to handle the case where the model install is initiated outside the app (e.g. via the new route).
  • Update the install confirmation page. There are 4 scenarios, 3 of which are failure cases:
    image
    image
    image
    image

@github-actions github-actions bot added the python-tests PRs that change python tests label Jun 17, 2024
@chainchompa chainchompa merged commit 70e40fa into main Jun 17, 2024
14 checks passed
@chainchompa chainchompa deleted the chainchompa/model-install-deeplink branch June 17, 2024 01:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api frontend PRs that change frontend files python PRs that change python files python-tests PRs that change python tests services PRs that change app services
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants