From da88552721e7932ca7fdd5e683aed3bd67c71a82 Mon Sep 17 00:00:00 2001 From: Tom Aarsen Date: Tue, 19 Nov 2024 13:27:05 +0100 Subject: [PATCH 1/2] Given that self.active_adapter is deprecated, avoid using it --- src/transformers/integrations/peft.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/transformers/integrations/peft.py b/src/transformers/integrations/peft.py index 8afff36eb086..9907e803f139 100644 --- a/src/transformers/integrations/peft.py +++ b/src/transformers/integrations/peft.py @@ -457,7 +457,7 @@ def get_adapter_state_dict(self, adapter_name: Optional[str] = None) -> dict: from peft import get_peft_model_state_dict if adapter_name is None: - adapter_name = self.active_adapter() + adapter_name = self.active_adapters()[0] adapter_state_dict = get_peft_model_state_dict(self, adapter_name=adapter_name) return adapter_state_dict From a2eca2b5e86ff3707521f65b746ec5952edd77b9 Mon Sep 17 00:00:00 2001 From: Tom Aarsen Date: Tue, 19 Nov 2024 13:27:31 +0100 Subject: [PATCH 2/2] Remove misleading comment - `self.active_adapter` is not used (and deprecated) --- src/transformers/integrations/peft.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/transformers/integrations/peft.py b/src/transformers/integrations/peft.py index 9907e803f139..b3352be0f950 100644 --- a/src/transformers/integrations/peft.py +++ b/src/transformers/integrations/peft.py @@ -381,7 +381,7 @@ def enable_adapters(self) -> None: If you are not familiar with adapters and PEFT methods, we invite you to read more about them on the PEFT official documentation: https://huggingface.co/docs/peft - Enable adapters that are attached to the model. The model will use `self.active_adapter()` + Enable adapters that are attached to the model. """ check_peft_version(min_version=MIN_PEFT_VERSION)