-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Feature: HiDiffusion integration #8787
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
base: main
Are you sure you want to change the base?
Feature: HiDiffusion integration #8787
Conversation
|
I might be spoiled by SDXL too much, but SD1.5 did not impress me much. It produces better results than default denoise. Especially when generating whole image. Inpainting is a tricky one. You might want to try different schedulers. Hands are particularly messy at 1.5k px. |
|
I just realised that updated version of HiDiffusion is here: https://github.com/Teriks/dgenerate/tree/master/dgenerate/extras/hidiffusion |
|
You may wish to consider vendoring the code inside the Invoke package, e.g. |
…ps for external dependency
|
I did include allegedly improved HiDiffusion in Invoke/backend/hidiffusion (for now, until we decide vendor it or not). |
The quality fixes were for checkpoint detection and deterministic generation, rather than adjusting core functionality. |
|
@DustyShoe It looks like line endings are off throughout your entire PR. Can you fix this? Try |
|
I already started to think that VSCode is playing tricks on me. Whew! |
Summary
Added HiDiffusion support across backend and UI. The denoise pipeline can now apply HiDiffusion with optional RAU‑Net and Window Attention toggles, including metadata persistence/recall and graph wiring for SD1/SDXL and the SDXL refiner. The UI exposes aligned advanced switches with informational popovers.
Added HiDiffusion documentation and a “Learn more” link.
Related Issues / Discussions
https://github.com/megvii-research/HiDiffusion
Closes #8780
QA Instructions
Merge Plan
Should merge without issue.
Checklist
What's Newcopy (if doing a release after this PR)