Skip to content

Conversation

@thedarkzeno
Copy link
Contributor

The mask and instance image were being cropped in different ways without --center_crop, causing the model to learn to ignore the mask in some cases. This PR fixes that and generate more consistent results.
toy_example

A thanks to @nathanielherman that pointed out the problem

The mask and instance image were being cropped in different ways without --center_crop, causing the model to learn to ignore the mask in some cases. This PR fixes that and generate more consistent results.
@HuggingFaceDocBuilderDev
Copy link

HuggingFaceDocBuilderDev commented Dec 2, 2022

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

@patrickvonplaten
Copy link
Contributor

Ok!

@patrickvonplaten patrickvonplaten merged commit e289998 into huggingface:main Dec 5, 2022
tcapelle pushed a commit to tcapelle/diffusers that referenced this pull request Dec 12, 2022
The mask and instance image were being cropped in different ways without --center_crop, causing the model to learn to ignore the mask in some cases. This PR fixes that and generate more consistent results.
sliard pushed a commit to sliard/diffusers that referenced this pull request Dec 21, 2022
The mask and instance image were being cropped in different ways without --center_crop, causing the model to learn to ignore the mask in some cases. This PR fixes that and generate more consistent results.
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.

3 participants