LCM Add Tests #5707
Merged
LCM Add Tests #5707
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
The documentation is not available anymore as the PR was closed or merged. |
sayakpaul
reviewed
Nov 9, 2023
src/diffusers/loaders.py
Outdated
Comment on lines
1205
to
1208
| state_dict, network_alphas = self.lora_state_dict( | ||
| pretrained_model_name_or_path_or_dict, | ||
| **kwargs, | ||
| ) |
There was a problem hiding this comment.
If we remove the last ","m this become one-line.
sayakpaul
reviewed
Nov 9, 2023
src/diffusers/loaders.py
Outdated
| @@ -1325,6 +1338,7 @@ def lora_state_dict( | |||
| weight_name = cls._best_guess_weight_name( | |||
| pretrained_model_name_or_path_or_dict, file_extension=".safetensors" | |||
| ) | |||
| print("weight_name", weight_name) | |||
sayakpaul
reviewed
Nov 9, 2023
src/diffusers/loaders.py
Outdated
Comment on lines
1454
to
1461
| if any(f.endswith(LORA_WEIGHT_NAME) for f in targeted_files): | ||
| targeted_files = list( | ||
| filter(lambda x: x.endswith(LORA_WEIGHT_NAME), targeted_files) | ||
| ) | ||
| elif any(f.endswith(LORA_WEIGHT_NAME_SAFE) for f in targeted_files): | ||
| targeted_files = list( | ||
| filter(lambda x: x.endswith(LORA_WEIGHT_NAME_SAFE), targeted_files) | ||
| ) |
sayakpaul
reviewed
Nov 9, 2023
src/diffusers/utils/peft_utils.py
Outdated
| @@ -204,6 +209,81 @@ def set_weights_and_activate_adapters(model, adapter_names, weights): | |||
| module.active_adapter = adapter_names | |||
|
|
|||
|
|
|||
| def find_adapter_config_file( | |||
sayakpaul
reviewed
Nov 9, 2023
src/diffusers/loaders.py
Outdated
Comment on lines
1674
to
1678
| if peft_config is not None: | ||
| lora_config_kwargs = LoraConfig.from_json_file(peft_config) | ||
| else: | ||
| lora_config_kwargs = get_peft_kwargs(rank, network_alphas, state_dict, is_unet=True) | ||
|
|
There was a problem hiding this comment.
Shouldn't this come under an if USE_PEFT_BACKEND?
There was a problem hiding this comment.
Good point!
|
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. |
kashif
pushed a commit
to kashif/diffusers
that referenced
this pull request
Nov 11, 2023
* lcm add tests * uP * Fix all * uP * Add * all * uP * uP * uP * uP * uP * uP * uP
yoonseokjin
pushed a commit
to yoonseokjin/diffusers
that referenced
this pull request
Dec 25, 2023
* lcm add tests * uP * Fix all * uP * Add * all * uP * uP * uP * uP * uP * uP * uP
AmericanPresidentJimmyCarter
pushed a commit
to AmericanPresidentJimmyCarter/diffusers
that referenced
this pull request
Apr 26, 2024
* lcm add tests * uP * Fix all * uP * Add * all * uP * uP * uP * uP * uP * uP * uP
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR adds tests for LCM