[wip] Introduce MultimethodLoraConfig#17230
[wip] Introduce MultimethodLoraConfig#17230lucylq wants to merge 2 commits intogh/lucylq/133/basefrom
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. ❌ 3 New Failures, 2 Unrelated FailuresAs of commit c918989 with merge base eac0673 ( NEW FAILURES - The following jobs have failed:
BROKEN TRUNK - The following jobs failed but were present on the merge base:👉 Rebase onto the `viable/strict` branch to avoid these failures
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.
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.
Stack from ghstack (oldest at bottom):
Differential Revision: D92315627