Skip to content

Fix broken Lora/Networks: use old method option#12466

Merged
AUTOMATIC1111 merged 1 commit intoAUTOMATIC1111:devfrom
catboxanon:fix/lora-old-mk2
Aug 11, 2023
Merged

Fix broken Lora/Networks: use old method option#12466
AUTOMATIC1111 merged 1 commit intoAUTOMATIC1111:devfrom
catboxanon:fix/lora-old-mk2

Conversation

@catboxanon
Copy link
Collaborator

@catboxanon catboxanon commented Aug 11, 2023

Description

Closes #12104

Fixes a regression introduced by b75b004 as part of the extra networks rework (#11821). As far as I can tell this was just an oversight during refactoring. This fixes the RuntimeError and produces an image identical to one before this regression.

Note that this function is only ever used when the option is enabled, so it doesn't affect the "new normal" for LoRA operation.

def network_forward(module, input, original_forward):
"""
Old way of applying Lora by executing operations during layer's forward.
Stacking many loras this way results in big performance degradation.
"""

Checklist:

@catboxanon catboxanon changed the title Fix to make Lora/Networks: use old method setting work Fix broken Lora/Networks: use old method option Aug 11, 2023
@AUTOMATIC1111 AUTOMATIC1111 merged commit 2c79f2a into AUTOMATIC1111:dev Aug 11, 2023
@w-e-w w-e-w mentioned this pull request Aug 24, 2023
@catboxanon catboxanon deleted the fix/lora-old-mk2 branch March 4, 2024 23:28
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.

[Bug]: Generating using LoRA fails with Runtime Error with Lora/Networks: use old method enabled

2 participants