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

feat: add enableImportFromHuggingFace option #2709

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

agatha197
Copy link
Contributor

@agatha197 agatha197 commented Sep 19, 2024

Add support for importing models from Hugging Face

This PR introduces a new feature that allows users to import models from Hugging Face. The changes include:

  • Added a new configuration option enableImportFromHuggingFace in config.toml.sample
  • Updated the BackendAIConfig type to include the new configuration option
  • Modified the VFolderListPage component to conditionally render the "Import from Hugging Face" button based on the new configuration
  • Updated the BackendAILogin component to handle the new configuration option and pass it to the global client configuration

Checklist:

  • Documentation: Update user documentation to explain the new feature and how to enable it
  • Minimum required manager version: Backend.AI 24.09
  • Test case: Verify that the "Import from Hugging Face" button appears when enableImportFromHuggingFace is set to true and is hidden when set to false

Copy link

graphite-app bot commented Sep 19, 2024

Your org requires the Graphite merge queue for merging into main

Add the label “flow:merge-queue” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “flow:hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

@github-actions github-actions bot added the size:S 10~30 LoC label Sep 19, 2024
@agatha197 agatha197 force-pushed the feature/add-import-from-hugging-face-modal branch from 77422f2 to f9c9efd Compare September 19, 2024 10:19
@agatha197 agatha197 force-pushed the feature/add-enableImportFromHuggingFace-option branch from 3b0ef99 to 96dce7b Compare September 19, 2024 10:20
Copy link
Member

@yomybaby yomybaby left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@agatha197 agatha197 force-pushed the feature/add-import-from-hugging-face-modal branch from f9c9efd to d57d172 Compare September 20, 2024 03:49
@agatha197 agatha197 force-pushed the feature/add-enableImportFromHuggingFace-option branch from 96dce7b to b4b1ee2 Compare September 20, 2024 03:49
Copy link

graphite-app bot commented Sep 20, 2024

Merge activity

# Add support for importing models from Hugging Face

This PR introduces a new feature that allows users to import models from Hugging Face. The changes include:

- Added a new configuration option `enableImportFromHuggingFace` in `config.toml.sample`
- Updated the `BackendAIConfig` type to include the new configuration option
- Modified the `VFolderListPage` component to conditionally render the "Import from Hugging Face" button based on the new configuration
- Updated the `BackendAILogin` component to handle the new configuration option and pass it to the global client configuration

**Checklist:**

- [ ] Documentation: Update user documentation to explain the new feature and how to enable it
- [x] Minimum required manager version: Backend.AI 24.09
- [x] Test case: Verify that the "Import from Hugging Face" button appears when `enableImportFromHuggingFace` is set to true and is hidden when set to false
@yomybaby yomybaby force-pushed the feature/add-import-from-hugging-face-modal branch from d57d172 to c333321 Compare September 20, 2024 03:59
@yomybaby yomybaby force-pushed the feature/add-enableImportFromHuggingFace-option branch from b4b1ee2 to be6db95 Compare September 20, 2024 04:00
Base automatically changed from feature/add-import-from-hugging-face-modal to main September 20, 2024 04:01
@graphite-app graphite-app bot merged commit be6db95 into main Sep 20, 2024
4 checks passed
@graphite-app graphite-app bot deleted the feature/add-enableImportFromHuggingFace-option branch September 20, 2024 04:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S 10~30 LoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants