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

EMA: support side-by-side validations with base model network #519

Closed
bghira opened this issue Jun 22, 2024 · 0 comments
Closed

EMA: support side-by-side validations with base model network #519

bghira opened this issue Jun 22, 2024 · 0 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers

Comments

@bghira
Copy link
Owner

bghira commented Jun 22, 2024

when training using EMA, the validations follow upstream Diffusers, and we temporarily overwrite the unet / transformer parameters in the base model with the EMA weights, before running inference.

however, we should provide the ability to generate the validations using both sets of weights, and stitch them together into a side by side representation, similar to this, with each output labeled:

image

@bghira bghira added documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers labels Jun 22, 2024
bghira added a commit that referenced this issue Aug 25, 2024
(#519) add side by side comparison with base model
@bghira bghira closed this as completed Aug 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant