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.
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
[LoRA] fix: lora loading when using with a device_mapped model. #9449
[LoRA] fix: lora loading when using with a device_mapped model. #9449
Changes from 22 commits
dc1aee2
949a929
64b3ad1
6d03c12
d4bd94b
5479198
2846549
1ed0eb0
d2d59c3
5f3cae2
8f670e2
e42ec19
f63b04c
eefda54
ea727a3
71989e3
f62afac
2334f78
5ea1173
f64751e
c0dee87
4b6124a
fe2cca8
2db5d48
61903c8
03377b7
0bd40cb
a61b754
ccd8d2a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
After-effects of
make fix-copies
.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.
@DN6 it would make sense to make this a separate utility instead of having redefine three times. WDYT?
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.
Yup, you can add as a util function inside pipeline_utils.
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.
Because for connected pipelines, we don't support device mapping in the first place.