-
-
Notifications
You must be signed in to change notification settings - Fork 112
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
Issue merging LoRA with latest version #371
Comments
I walked the commits back and the issue first appeared with this commit: |
same here in "stable diffusion forge" |
I am having the same issue, I believe it has something to do with the diffusers version! but i could be wrong. |
I'm running into the same issue. Are you trying to merge SDXL LoRAs? I suspect it is related to the conditioning. Ain't |
+1 |
A couple of temporary workarounds to get XL LoRA merging capability back: Reset to the commit before the bug was introduced:
Or clone this temporary fork of the repo without the above issue (No fix, just pinned to the commit before the problem was introduced): Either way, you'll lose a few commits worth of updates, but merging capability is restored to a known-good commit. |
This worked for me, a temp fix but made merging XL loras work again. now I just have to remember not to update! Thank you n15g |
Going back using git reset --hard 97030c8 is not working for merging SDXL LoRA for me anymore after I updated my Automatic1111. It used to work on 1.8... Edit: OK it was using the wrong lora name. Now it worked... sorry. But it still needs to be on that old commit. |
No fix? |
working in forge after git reset --hard 97030c8 |
The XL LoRAs have merged so far have not caused any issues. |
I updated the plugin recently and have found that attempts to merge any LoRA now throw the following IndexError:
I added a debug output to find out what the problematic index is and it seems to be related to the switchover to the second text encoder. As soon as the second text encoder layers get hit, the index returns 27 and overflows the length of the
ratios
array.That's as far as I've managed to get, as I'm still familiarizing myself with the code.
From
pluslora.py:483
:The text was updated successfully, but these errors were encountered: