-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[fea] moe support #8498
[fea] moe support #8498
Conversation
Thanks for your contribution! |
kebo01 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
0e4c89d
to
55dd5a3
Compare
92dbe7c
to
2851da8
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #8498 +/- ##
===========================================
- Coverage 54.27% 53.97% -0.31%
===========================================
Files 617 618 +1
Lines 96356 96801 +445
===========================================
- Hits 52302 52249 -53
- Misses 44054 44552 +498 ☔ View full report in Codecov by Sentry. |
939c0d2
to
38dfa82
Compare
f0cde72
to
67981e5
Compare
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.
LGTM
有些代码看看能不能是不是参考我这个PR的修改:#8511 |
28e8dcc
to
813c705
Compare
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.
LGTM
Co-authored-by: kebo01 <[email protected]>
* [fea] moe support (#8498) Co-authored-by: kebo01 <[email protected]> * [fix] Broadcast optimizer state using broadcast_dp without shard-reshard. (#8522)
PR types
PR changes
Description