Feature: Support of Manifold Hyper Connection(mHC)#3430
Feature: Support of Manifold Hyper Connection(mHC)#3430jingqiny-99 wants to merge 10 commits intoNVIDIA:mainfrom
Conversation
a1d8ca5 to
1e66788
Compare
…onnection(mHC). (NVIDIA#2943) Co-authored-by: Jingqin Yang <jingqiny@login-eos01.eos.clusters.nvidia.com> Co-authored-by: root <root@eos0478.eos.clusters.nvidia.com> Co-authored-by: Dennis Liu <denliu@nvidia.com>
e7e1a13 to
62ea06f
Compare
|
/claude review |
|
/ok to test 6a1e311 |
|
/ok to test 35b1bec |
|
/ok to test 9b1e544 |
|
/ok to test 3431dab |
|
Should this PR remain open given #3828? |
#3828 was a follow up optimization on top of original PR. since it is kernel only and self contained, @jingqiny-99 has since merged it together into this PR for main. It is final and ready for review now |
|
@jingqiny-99 this PR is quite large and I think would benefit from being split into multiple smaller PRs. A new Claude skill |
Sure, I'll try it. |
What does this PR do ?
mirror PR of #2943
Contribution process
flowchart LR A[Pre-checks] --> B[PR Tests] subgraph Code Review/Approval C1[Expert Review] --> C2[Final Review] end B --> C1 C2 --> D[Merge]Pre-checks
Core 0.8)Code review
The following process is enforced via the CODEOWNERS file for changes into
megatron/core. For changes outside ofmegatron/core, it is up to the PR author whether or not to tag the Final Reviewer team.For MRs into `main` branch
Feel free to message or comment the @mcore-oncall to help accelerate your merge into main. The less complex your PR is, the faster it will be approved and merged!
(Step 1): Add PR label
Expert Review(Step 2): Collect the expert reviewers reviews
Expert Reviewlabel when your PR is ready for review.Final Review might get declined if these requirements are not fulfilled.
(Step 3): Final Review
Final Reviewlabel(Optional Step 4): Cherry-pick into release branch
If this PR also needs to be merged into
core_r*release branches, after this PR has been merged, selectCherry-pickto open a new PR into the release branch.For MRs into `dev` branch
The proposed review process for `dev` branch is under active discussion.MRs are mergable after one approval by either
eharper@nvidia.comorzijiey@nvidia.com.Merging your PR
Any member of core-adlr and
core-nemowill be able to merge your PR.