Skip to content

Improve experience for cv models#408

Merged
luv-bansal merged 3 commits intomasterfrom
cv_ux
Oct 8, 2024
Merged

Improve experience for cv models#408
luv-bansal merged 3 commits intomasterfrom
cv_ux

Conversation

@luv-bansal
Copy link
Copy Markdown
Contributor

@luv-bansal luv-bansal commented Oct 7, 2024

What

  • If no concepts are defined in the YAML file and the download_checkpoints flag is set to false, the user is prompted to download the checkpoints to load the concepts. Once the concepts are defined in config.yaml, the process continues with creating the tar file, but without including the checkpoints

Why

  • Currently, for models that require concepts, an error is thrown if no concepts are defined in the YAML file, resulting in a poor user experience when uploading the model.

How

Tests

Notes

@luv-bansal luv-bansal requested a review from zeiler October 7, 2024 08:33
Comment thread clarifai/runners/models/model_upload.py Outdated
@luv-bansal luv-bansal requested a review from zeiler October 8, 2024 08:12
@luv-bansal luv-bansal enabled auto-merge (squash) October 8, 2024 08:18
Comment thread clarifai/runners/models/model_upload.py Outdated
@luv-bansal luv-bansal requested a review from zeiler October 8, 2024 16:16
@luv-bansal luv-bansal merged commit 8469c39 into master Oct 8, 2024
@luv-bansal luv-bansal deleted the cv_ux branch October 8, 2024 16:49
@luv-bansal luv-bansal mentioned this pull request Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants