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

[Community Pipeline] Checkpoint Merger based on Automatic1111 #1472

Merged
merged 6 commits into from Dec 5, 2022
Merged

[Community Pipeline] Checkpoint Merger based on Automatic1111 #1472

merged 6 commits into from Dec 5, 2022

Conversation

Abhinay1997
Copy link
Contributor

The checkpoint merger pipeline based on the discussion at #877.

Tested the checkpoint merger for the case with two checkpoints.

@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Nov 29, 2022

The documentation is not available anymore as the PR was closed or merged.

@Abhinay1997
Copy link
Contributor Author

@patrickvonplaten could you have a look at this once you're free ?

@patrickvonplaten
Copy link
Contributor

This looks very cool! Could you add an entry to the table here and also add a section of how to use your community pipeline? :-)

https://github.com/huggingface/diffusers/blob/main/examples/community/README.md#community-examples

@hafriedlander
Copy link
Contributor

@Abhinay1997 is this a clean redevelopment, or does it use code from the Automatic1111 repository?

Unfortunately Automatic1111 (deliberately) does not have a license for his code, and it can't be legally used in other projects.

@Abhinay1997
Copy link
Contributor Author

@hafriedlander, only the interpolation functions are from Automatic1111 but they are originally from Wikipedia. Last I checked, Automatic1111 had changed their code in the current version.

@Abhinay1997
Copy link
Contributor Author

Abhinay1997 commented Dec 2, 2022

I was using https://github.com/AUTOMATIC1111/stable-diffusion-webui/blob/fdecb636855748e03efc40c846a0043800aadfcc/modules/extras.py as reference. Basic interpolation functions are strictly math equations right ? Everything else i.e loading the checkpoints, comparing the attributes and running the merged pipeline is all a clean rewrite.

What do you think @patrickvonplaten ?

@Abhinay1997
Copy link
Contributor Author

Abhinay1997 commented Dec 2, 2022

Meanwhile, I have added the docs in the README as suggested Patrick.

@patrickvonplaten
Copy link
Contributor

Thanks!

@patrickvonplaten patrickvonplaten merged commit c28d694 into huggingface:main Dec 5, 2022
tcapelle pushed a commit to tcapelle/diffusers that referenced this pull request Dec 12, 2022
…gface#1472)

* Add checkpoint_merger pipeline

* Added missing docs for a parameter.

* Fomratting fixes.

* Fixed code quality issues.

* Bug fix: Off by 1 index

* Added docs for pipeline
sliard pushed a commit to sliard/diffusers that referenced this pull request Dec 21, 2022
…gface#1472)

* Add checkpoint_merger pipeline

* Added missing docs for a parameter.

* Fomratting fixes.

* Fixed code quality issues.

* Bug fix: Off by 1 index

* Added docs for pipeline
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

4 participants