Skip to content

Commit

Permalink
Fix Unipelt Lora default config (#682)
Browse files Browse the repository at this point in the history
Fixes #674.
  • Loading branch information
calpt authored Apr 17, 2024
1 parent 2652d27 commit a3545c5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/method_combinations.md
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ which is identical to the following `ConfigUnion`:
from adapters import ConfigUnion, LoRAConfig, PrefixTuningConfig, SeqBnConfig

config = ConfigUnion(
LoRAConfig(r=8, use_gating=True),
LoRAConfig(r=8, alpha=2, use_gating=True),
PrefixTuningConfig(prefix_length=10, use_gating=True),
SeqBnConfig(reduction_factor=16, use_gating=True),
)
Expand Down
2 changes: 1 addition & 1 deletion src/adapters/configuration/adapter_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -621,7 +621,7 @@ def __init__(
components = [
prefix_tuning or PrefixTuningConfig(prefix_length=10),
adapter or SeqBnConfig(reduction_factor=16),
lora or LoRAConfig(r=8),
lora or LoRAConfig(r=8, alpha=2),
]

super().__init__(*[c.replace(use_gating=True) for c in components])
Expand Down

0 comments on commit a3545c5

Please sign in to comment.