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

DoRA support #608

Closed
wants to merge 1 commit into from
Closed

DoRA support #608

wants to merge 1 commit into from

Conversation

catboxanon
Copy link
Collaborator

@catboxanon catboxanon commented Mar 27, 2024

Title. Adapted from comfyanonymous/ComfyUI@ae77590


Semi-related: I don't think my question I originally asked in #80 was addressed well. Are we planning to sync upstream of both the webui and ComfyUI? Or, only the webui, and to cherry-pick certain updates from ComfyUI such as this one?

@GavChap
Copy link

GavChap commented Apr 10, 2024

I've checked this locally by pulling the request branch and it works fine, will this be merged at any point?

@ntc-ai
Copy link

ntc-ai commented Apr 19, 2024

Hi, this matches ComfyUI but differs from sdwebui. The generated image changes at scale 0. I opened a patch to ComfyUI to try to standardize on sdwebui's approach:
comfyanonymous/ComfyUI#3298

@kleineluka
Copy link

@catboxanon can you change this pr to dev2 and could @huchenlei merge? seems like main is dead for now

@catboxanon catboxanon marked this pull request as draft May 29, 2024 21:13
Panchovix pushed a commit to Panchovix/stable-diffusion-webui-reForge that referenced this pull request Jun 30, 2024
@kusobabaa29 kusobabaa29 mentioned this pull request Jul 27, 2024
@lllyasviel lllyasviel marked this pull request as ready for review August 1, 2024 19:54
@lllyasviel
Copy link
Owner

hi we are going to close PRs before forge's recent major revision
if we missed some important PRs, please consider reopen (if that is not already on our todo list

@lllyasviel lllyasviel closed this Aug 1, 2024
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.

None yet

8 participants