-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Fused MOE for Mixtral #2542
Merged
Merged
Fused MOE for Mixtral #2542
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
c33e37a
Fused MOE for Mixtral
pcmoritz 03f5dc8
fix weight loading
pcmoritz 2867f34
lint and cleanup
pcmoritz a82f73f
style
pcmoritz 406a188
yapf
pcmoritz 707479d
update comment
pcmoritz 649287e
Merge branch 'main' into fused-mixtral
pcmoritz f283c98
update fused_moe implementation
pcmoritz adeb80f
optimize block sizes
pcmoritz d6a0b11
Merge branch 'main' into fused-mixtral
pcmoritz 74308ba
Merge branch 'main' into fused-mixtral
pcmoritz 23284e3
add dtype
pcmoritz 429f51c
Merge branch 'main' into fused-mixtral
pcmoritz a1f0107
update
pcmoritz f1691ba
Merge branch 'fused-mixtral' of https://github.com/pcmoritz/vllm-publ…
pcmoritz ee0a490
Merge branch 'main' into fused-mixtral
WoosukKwon a77a2c1
Resolve merge error
WoosukKwon 2233c2a
remove extra function
pcmoritz 44cc7d0
update
pcmoritz 7b076aa
move mixtral moe layer back
pcmoritz b4cb78c
put import back
pcmoritz 111c1b5
yapf
pcmoritz d30e844
fix typo
pcmoritz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
lint and cleanup
- Loading branch information
commit 2867f346274c1f850597a42f447b2ef7e4266592
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
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.
@pcmoritz Should we move the
MoE
class back to the Mixtral model file? It seems like this MoE layer is not shared between Mixtral and DeepSeek.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.
Sounds good to me! Feel free to make any edits to the PR you'd like to make or let me know if I should make them :)
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.
I'd appreciate it if you can do it!