-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[WIP] Update LoraConfig for KaSA implementation
#2698
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
iambogeumkim
wants to merge
40
commits into
huggingface:main
Choose a base branch
from
iambogeumkim:peft-kasa
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
387ad2a
Add KaSA implementation to layer.py
iambogeumkim ae00a34
Add `use_kasa` argument to LoraConfig
iambogeumkim 6588f1a
Add use_kasa parameter to Linear class
iambogeumkim a824ac9
Add KasaLinearVariant class (just copy of DoraLinearVariant class) in…
iambogeumkim 05e4e07
Add kasa description
iambogeumkim d1e7e43
Remove unnecessary self.kasa
iambogeumkim 9e53b9f
[WIP] update KasaLinearVariant class with SVD implementation
iambogeumkim aa37111
Modify merge/unmerge method in KasaLinearVariant class
iambogeumkim 9cfe65c
update KasaLinearVariant class with SVD implementation
iambogeumkim f9d7cc7
fix type in init method
iambogeumkim 84813a3
delete unnecessary part in layer.py
iambogeumkim 39abcad
add original reference in layer.py
iambogeumkim 06f76d8
merge main to peft-kasa
iambogeumkim 0043ae3
re-add KaSA implementation to variants.py
iambogeumkim ea59432
add use_kasa param to resolve_lora_variant in other layers
iambogeumkim 574c1b8
delete unnecessary part in layer.py
iambogeumkim 73fa58f
delete unnecessary part in variants.py
iambogeumkim b9e3190
add _get_delta_weight static method in KasaLinearVariants class
iambogeumkim 2649fdf
update module.get_delta_weight to KasaLinearVariant._get_delta_weight…
iambogeumkim d06cafd
add kasa test
iambogeumkim 4ee5da1
add dropout
iambogeumkim 0377170
Update tests/test_custom_models.py
iambogeumkim a536bbf
Update src/peft/tuners/lora/variants.py
iambogeumkim f8d8057
Update src/peft/tuners/lora/variants.py
iambogeumkim cd57c7b
Update src/peft/tuners/lora/variants.py
iambogeumkim 2431a2c
add use_kasa param in LoraModel class
iambogeumkim 7276b3b
restore output_tensor variable in Linear class get_delta_weight method
iambogeumkim 5a67b1f
add use_kasa handling condition in resolve_lora_variant method
iambogeumkim 3ec6b18
fix KaSA self, mat1, mat2 dtype error
iambogeumkim 461a89c
fix make style error
iambogeumkim 6ed64c1
add _skip_test_disable_adapters function
iambogeumkim 92061dd
add KaSA compatibility test
iambogeumkim 9a2cf71
update _check_new_adapter_config method with KaSA
iambogeumkim 39cf1f9
Implement tests to ensure KaSA adapters cannot be mixed with other ad…
iambogeumkim 283ff0a
Refactor KaSA adapter compatibility check to simplify logic and impro…
iambogeumkim bfe8996
Refactor KasaLinearVariant class to improve code readability and ensu…
iambogeumkim cbc5b0c
Merge branch 'main' into peft-kasa
iambogeumkim 14fa9d7
Merge branch 'main' into peft-kasa
iambogeumkim b6aae1e
Remove tests for mixing KaSA adapters with other adapter types in Tes…
iambogeumkim 951b6b2
Add tests to validate that KaSA adapters cannot be mixed with other a…
iambogeumkim File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's remove this and call
super()._check_new_adapter_config(config)instead.