Skip to content
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

Add PEFT add_weighted_adapter() Function for Merging Multiple Adapters #6310

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Dlemonha
Copy link

What does this PR do?

The original way to merge multiple Lora adapters is to use the for loop merge_and_unload() to add to the base model. Here, the add_weighted_adapter() method of PEFT is added to enhance the effect of merging multiple adapters.

Fixes # (issue)

Before submitting

zhengyaowei and others added 5 commits December 6, 2024 05:14
The original way to merge multiple Lora adapters is to use the for loop merge_and_unload() to add to the base model. Here, the add_weighted_adapter() method of PEFT is added to enhance the effect of merging multiple adapters.
@hiyouga hiyouga added the pending This problem is yet to be addressed label Dec 11, 2024
@Dlemonha
Copy link
Author

Hi, do you think this method is helpful for the project? @hiyouga

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending This problem is yet to be addressed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants