-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Better warning message or error when target module names are not available #40
Comments
Above PR should resolve this issue by raise error. For the example you specified in the issue description, following is the error message that will be raised
|
I know maybe it's a different question but how do I solve this if I get this error? How to know what target module name I need? |
hi @nameless0704, did you get any solution for the error? |
Hello, for models that aren't in the https://github.com/huggingface/peft/blob/main/src/peft/utils/other.py#L202, you need to look into the |
for example, you can or |
Can you help me understand what parameter will it be in this model, I can see there is nothing like |
We have documented how to apply adapters to models that are not pre-configured here. Moreover, you can pass |
If we do:
This wouldn't have any effect and will lead to errors during training because the
target_modules
were incorrectly named insideLoraConfig
. When this case arises, it would help to throw a warning message so that the users can look into the issue with more information.The text was updated successfully, but these errors were encountered: