-
Notifications
You must be signed in to change notification settings - Fork 837
Introduce MultimethodLoraConfig #17230
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
base: gh/lucylq/133/base
Are you sure you want to change the base?
Conversation
Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) [ghstack-poisoned]
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/17230
Note: Links to docs will display an error until the docs builds have been completed. ❌ 16 New Failures, 1 Cancelled JobAs of commit 9c941c7 with merge base efe4f0c ( NEW FAILURES - The following jobs have failed:
CANCELLED JOB - The following job was cancelled. Please retry:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) [ghstack-poisoned]
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.
Pull request overview
This PR introduces MultimethodLoraConfig, a new configuration class that enables exporting multiple methods (e.g., base model and LoRA variants) to a single .pte file. This is part of a larger stack adding multimethod support to the LLM export pipeline.
Changes:
- Added
Dictto typing imports to support the new configuration structure - Introduced
MultimethodLoraConfigclass with a dictionary mapping method names to optional LoRA configurations - Integrated
MultimethodLoraConfigas a new field in theLlmConfigclass
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) [ghstack-poisoned]
Stack from ghstack (oldest at bottom):
Differential Revision: D92315627