Skip to content

Feature: Support for resizing multiple format loras.#2057

Closed
Symbiomatrix wants to merge 3 commits intokohya-ss:mainfrom
Symbiomatrix:bugfix2
Closed

Feature: Support for resizing multiple format loras.#2057
Symbiomatrix wants to merge 3 commits intokohya-ss:mainfrom
Symbiomatrix:bugfix2

Conversation

@Symbiomatrix
Copy link
Contributor

Various trainers use different naming conventions for loras, which break the resize script. A ubiquitous one is lora_A / lora_B instead of lora_down / lora_up. This change reads whichever convention was used in the given model and adjusts all hardcoded references to that name.
A bit of a patchwork fix using global variables since I don't know the full flow of the code, but I've tested it extensively over several types of base models' loras (from 1.5 to wan) and it works for every lora I've tested.

@woct0rdho
Copy link
Contributor

In f-strings, we need to replace LORAFMT[0] and LORAFMT[1] with {LORAFMT[0]} and {LORAFMT[1]}

@Symbiomatrix
Copy link
Contributor Author

Symbiomatrix commented Jun 21, 2025

@woct0rdho Right, oops. Fixed. Eyeballing refactors in PR versions, bad habit of mine.

@woct0rdho woct0rdho mentioned this pull request Aug 14, 2025
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.

2 participants