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

Colab: added the ability to download models from civitai #1758

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

uptotec
Copy link

@uptotec uptotec commented Jan 5, 2024

its hard and time consuming to download models and putting them in the right directory in google colab so I added the ability to download checkpoints and LoRAs from civitai using model URL. the code uses civitai api to get model info and downloads it in the right directory.
please let me know if this change is acceptable and if any changes is required.

@mashb1t
Copy link
Collaborator

mashb1t commented Jan 28, 2024

Civitai now requires token auth for automated downloads, see #2041 (comment)

@uptotec
Copy link
Author

uptotec commented Feb 4, 2024

It works with me without issue. Please give it a try and tell me if any problem arise.

Copy link
Collaborator

@mashb1t mashb1t left a comment

Choose a reason for hiding this comment

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

Thanks for the bump, seems like Civitai has removed the global restrictions for auth again... 🤷
Please check the grammar in the provided code, see suggestions.
The code works fine but i am sadly not authorized to merge this PR.

fooocus_colab.ipynb Outdated Show resolved Hide resolved
fooocus_colab.ipynb Outdated Show resolved Hide resolved
fooocus_colab.ipynb Outdated Show resolved Hide resolved
added a check to make sure that the downloaded model is SDXL
@uptotec
Copy link
Author

uptotec commented Feb 5, 2024

made the required changes and added a check to make sure the downloaded model is SDXL.

Copy link
Collaborator

@mashb1t mashb1t left a comment

Choose a reason for hiding this comment

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

While i understand the reasoning behind this code we still have to consider that SD1.5 models can be used as refiners. I'd propose to keep the code but please keep this in mind when reviewing.

@uptotec
Copy link
Author

uptotec commented Feb 5, 2024

this code is for colab and colab has the refiner disabled by default.

@mashb1t mashb1t added the Size M medium change, isolated, testing with care label Feb 9, 2024
@mashb1t
Copy link
Collaborator

mashb1t commented Mar 3, 2024

@uptotec please check out #2423 and feel free to provide your feedback / learnings

@Chandima1990
Copy link

Hi is this going to be merged? @lllyasviel?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Size M medium change, isolated, testing with care
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants