Introduce MultimethodLoraConfig#17230
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. ⏳ No Failures, 8 PendingAs of commit e619591 with merge base aa2f683 ( 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.
Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) [ghstack-poisoned]
Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) [ghstack-poisoned]
Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) [ghstack-poisoned]
Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) [ghstack-poisoned]
470fce2
into
gh/lucylq/133/base
This PR was created by the merge bot to help merge the original PR into the main branch. ghstack PR number: #17230 by @lucylq ^ Please use this as the source of truth for the PR details, comments, and reviews ghstack PR base: https://github.com/pytorch/executorch/tree/gh/lucylq/133/base ghstack PR head: https://github.com/pytorch/executorch/tree/gh/lucylq/133/head Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/lucylq/132/orig Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/lucylq/133/orig Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) @diff-train-skip-merge --------- Co-authored-by: Github Executorch <github_executorch@arm.com> Co-authored-by: lucylq <lfq@meta.com>
This PR was created by the merge bot to help merge the original PR into the main branch. ghstack PR number: pytorch#17230 by @lucylq ^ Please use this as the source of truth for the PR details, comments, and reviews ghstack PR base: https://github.com/pytorch/executorch/tree/gh/lucylq/133/base ghstack PR head: https://github.com/pytorch/executorch/tree/gh/lucylq/133/head Merge bot PR base: https://github.com/pytorch/executorch/tree/gh/lucylq/132/orig Merge bot PR head: https://github.com/pytorch/executorch/tree/gh/lucylq/133/orig Differential Revision: [D92315627](https://our.internmc.facebook.com/intern/diff/D92315627/) @diff-train-skip-merge --------- Co-authored-by: Github Executorch <github_executorch@arm.com> Co-authored-by: lucylq <lfq@meta.com>
Stack from ghstack (oldest at bottom):
Differential Revision: D92315627