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

[WIP] Add Fabric #4201

Merged
merged 104 commits into from Aug 28, 2023
Merged

[WIP] Add Fabric #4201

merged 104 commits into from Aug 28, 2023

Conversation

shauray8
Copy link
Contributor

@shauray8 shauray8 commented Jul 21, 2023

What does this PR do?

Adds Fabric non-training-based iterative image improvement to diffusion pipeline

code - https://github.com/sd-fabric/fabric
paper - https://huggingface.co/papers/2307.10159
Fixes #4173

Before submitting

Test Results

Without Feedback With Feedback (1st image)
Image 1 Feedback Image 1

Who can review?

@patrickvonplaten

@shauray8
Copy link
Contributor Author

Author implementation at https://github.com/sd-fabric/fabric seems pretty straightforward, reading the paper (https://arxiv.org/abs/2307.10159) for more insights!

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Copy link
Member

@stevhliu stevhliu 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 iterating on this! A few more small fixes and the docs look good to me 😄

docs/source/en/api/pipelines/fabric.md Outdated Show resolved Hide resolved
docs/source/en/api/pipelines/fabric.md Outdated Show resolved Hide resolved
Copy link
Member

@sayakpaul sayakpaul left a comment

Choose a reason for hiding this comment

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

Looks good to me. Thanks a lot for iterating!

Let's maybe also reach out to the FABRIC authors once the PR is merged so that they are aware of the integration!

@shauray8
Copy link
Contributor Author

Looks good to me. Thanks a lot for iterating!

Let's maybe also reach out to the FABRIC authors once the PR is merged so that they are aware of the integration!

sounds good to me

@shauray8
Copy link
Contributor Author

unrelated tests failing - FAILED tests/pipelines/paint_by_example/test_paint_by_example.py

@patrickvonplaten
Copy link
Contributor

Hey @shauray8,

Thanks a lot for your PR! Can we maybe put this pipeline in the community folder: https://github.com/huggingface/diffusers/tree/main/examples/community?

We're sadly a bit overwhelmed by maintenance and aren't able to keep our slow tests clean - if we put this pipeline in the community folder, you can be the official author and the burden to maintain the pipeline is loosened. Would that be ok for you?

@shauray8
Copy link
Contributor Author

Okay, I think it's a good decision, I'll do it before EOD.

@shauray8
Copy link
Contributor Author

@sayakpaul @patrickvonplaten is there anything more I should remove or add?

specific language governing permissions and limitations under the License.
-->

# FABRIC
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we maybe also add this to the README.md in examples/community? :-)

Comment on lines 217 to 218
- local: api/pipelines/fabric
title: FABRIC
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- local: api/pipelines/fabric
title: FABRIC

@shauray8
Copy link
Contributor Author

It should look good now!

@patrickvonplaten
Copy link
Contributor

Amazing job @shauray8 ! The PR looks very clean now - thanks for bearing with us on this one!

@patrickvonplaten patrickvonplaten merged commit fd35689 into huggingface:main Aug 28, 2023
9 checks passed
@shauray8
Copy link
Contributor Author

Thank you @sayakpaul @patrickvonplaten and @stevhliu for all the reviews.

AmericanPresidentJimmyCarter pushed a commit to AmericanPresidentJimmyCarter/diffusers that referenced this pull request Apr 26, 2024
* empty PR

* init

* changes

* starting with the pipeline

* stable diff

* prev

* more things, getting started

* more functions

* makeing it more readable

* almost done testing

* var changes

* testing

* device

* device support

* maybe

* device malfunctions

* new new

* register

* testing

* exec does not work

* float

* change info

* change of architecture

* might work

* testing with colab

* more attn atuff

* stupid additions

* documenting and testing

* writing tests

* more docs

* tests and docs

* remove test

* empty PR

* init

* changes

* starting with the pipeline

* stable diff

* prev

* more things, getting started

* more functions

* makeing it more readable

* almost done testing

* var changes

* testing

* device

* device support

* maybe

* device malfunctions

* new new

* register

* testing

* exec does not work

* float

* change info

* change of architecture

* might work

* testing with colab

* more attn atuff

* stupid additions

* documenting and testing

* writing tests

* more docs

* tests and docs

* remove test

* change cross attention

* revert back

* tests

* reverting back to orig

* changes

* test passing

* pipeline changes

* before quality

* quality checks pass

* remove print statements

* doc fixes

* __init__ error something

* update docs, working on dim

* working on encoding

* doc fix

* more fixes

* no more dependent on 512*512

* update docs

* fixes

* test passing

* remove comment

* fixes and migration

* simpler tests

* doc changes

* green CI

* changes

* more docs

* changes

* new images

* to community examples

* selete

* more fixes

* changes

* fix

---------

Co-authored-by: Sayak Paul <spsayakpaul@gmail.com>
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.

FABRIC non-training based iterative image improvement
5 participants