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

Fix loading UNet only checkpoints #7523

Closed
wants to merge 1 commit into from

Conversation

CCRcmcpe
Copy link
Contributor

@CCRcmcpe CCRcmcpe commented Feb 4, 2023

Describe what this pull request is trying to achieve.

As in the title.
Some examples:
https://huggingface.co/Crosstyan/BPModel/blob/main/bp_mk5.safetensors
https://huggingface.co/AnnihilationOperator/ABPModel/tree/main

@CCRcmcpe
Copy link
Contributor Author

CCRcmcpe commented Feb 4, 2023

I did write a issue with all the backgrounds described but Github give me a f**king CONNECTION_CLOSED when I'm submitting it. Feel free if any questions.

@fractal-fumbler
Copy link

fixing my issue #6768

@AUTOMATIC1111
Copy link
Owner

I'd rather not encourage people sharing incomplete checkpoints. In any case, this doesn't get in as it is no matter what - disabling the optimization for everyone. You have the state dict at the moment when you create the model, so you can check if needed layers exist and disable stuff there on that condition.

But autodetection of SD2/SD1 config works using the text layer, and if it's missing, UI won't be able to differentiate between SD1/SD2 checkpoints, so again, I'd just rather not support people's incomplete checkpoints.

@AUTOMATIC1111
Copy link
Owner

pushed a commit that should make it possible to load load the checkpoints without disabling the optimization for everyone.

@CCRcmcpe CCRcmcpe closed this Feb 5, 2023
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.

None yet

3 participants