-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
[refactor] enhance readability of flux related pipelines #9711
Conversation
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice clean up! 👍🏽
@Luciennnnnnn can you run |
Sure! This is my first contribution to diffusers, i'll try it |
Done! |
The previous commit is failed on two checks, after modification, how to rerun checks |
Hi @yiyixuxu , I see checks failed on FLUX training examples, which is not related to my PR. How can I pass it? |
Hi @Luciennnnnnn thanks for the PR! It is a really nice PR
Would you be interested in working on that? If not, I can take over when I have a little bit more time:) |
Sure! I have updated training example accordingly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @Luciennnnnnn
really nice job:) hope to see more of your PRs soon
* flux pipline: readability enhancement.
What does this PR do?
This PR refactors the FLUX-related pipelines to improve clarity and consistency, particularly in regards to height and width determination. The changes align the implementation more closely with other pipelines in the library, enhancing overall code coherence and maintainability.
The original code obscured the details of how latents packing affects resolution, but this change makes the process more transparent and easier to understand.
@yiyixuxu @asomoza