Repository navigation
feat(nemotron): Add GTP + partial CG + cuDNN LN support for NT3 Ultra MXFP8 GB200/GB300/VRNVL72 recipes - #6228
Merged
malay-nagda merged 18 commits intoOct 13, 2026
Conversation
added 5 commits
September 26, 2026 11:10
…nd GB300 - Introduced `_apply_nemotron_3_ultra_gtp` function to configure GTP settings for dense weight sharding. - Updated `nemotron_3_ultra_pretrain_256gpu_gb200_fp8mx_config` and `nemotron_3_ultra_pretrain_256gpu_gb300_fp8mx_config` to apply GTP settings. - Added support for GTP sharding in the `moe_latent_proj` module to address memory pressure. Signed-off-by: Ravi Ghadia <rghadia@nvidia.com>
…into rghadia/nt3-ultra-gtp
Signed-off-by: Ravi Ghadia <rghadia@nvidia.com>
Signed-off-by: Ravi Ghadia <rghadia@nvidia.com>
- Partial CUDA graph for NT3 Ultra MXFP8 GB200/GB300 recipes - Added GTP sharding for the `moe_latent_proj` module in Nemotron 3 Ultra GB200 configuration. - Implemented local CUDA graph handling in training and validation processes. - Updated tests to ensure proper validation of local per-layer CUDA graph modules. - Improved error handling for unsupported local whole-layer captures. Signed-off-by: Ravi Ghadia <rghadia@nvidia.com>
…into rghadia/nt3-ultra-gtp-cg
…into rghadia/nt3-ultra-gtp-cg Signed-off-by: rghadia <rghadia@nsc-lhr-slurm-1-vscode-01.nvidia.com>
ghadiaravi13
marked this pull request as ready for review
October 9, 2026 23:04
ghadiaravi13
requested review from
dingqingy-nv and
malay-nagda
as code owners
October 9, 2026 23:04
- Changed references from HybridEP to NCCLEP in both GB200 and GB300 configurations. - Added support for cuDNN normalization kernels in both configurations. - Enabled NCCLEP and adjusted expert token handling in GB200. - Commented out the HSDP/FSDP dtype overrides in GB300 for clarity. These changes enhance the performance and compatibility of the Nemotron 3 Ultra pretraining setups. Signed-off-by: rghadia <rghadia@nvidia.com>
malay-nagda
previously approved these changes
Oct 10, 2026
Contributor
|
/ok to test 24cb4f9 |
Contributor
|
Automatic Claude reviews have been retired. To request a pull-request review, post a comment containing: Add |
Signed-off-by: rghadia <rghadia@nvidia.com>
…00 recipe Signed-off-by: rghadia <rghadia@nsc-lhr-slurm-1-vscode-01.nvidia.com>
Contributor
Author
|
/ok to test 39c25c6 |
@ghadiaravi13, there was an error processing your request: See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/2/ |
Contributor
Author
|
/ok to test 39c25c6 |
…into rghadia/nt3-ultra-gtp-cg
- Replaced HybridEP references with NCCLEP in GB300 and Rubin configurations. - Moved NCCLEP to the base recipe constructor so that Rubin recipe can pick up the NCCLEP changes - Updated test cases to reflect changes in execution layout and environment variables. - Improved clarity in comments regarding NCCL EP settings. These modifications enhance the performance and compatibility of the Nemotron 3 Ultra pretraining setups. Signed-off-by: rghadia <rghadia@nvidia.com>
Contributor
Author
|
/ok to test 892a800 |
- Removed "moe_router" from the set_cuda_graph_modules function calls in both GB200 and GB300 configurations. - Added a TODO comment to indicate the need for this change due to MCore PR 7046, which affects CPU-to-GPU copy for per token loss. Signed-off-by: rghadia <rghadia@nvidia.com>
Signed-off-by: Ravi Ghadia <rghadia@nvidia.com>
Contributor
|
/ok to test b892a0e |
malay-nagda
previously approved these changes
Oct 13, 2026
Contributor
|
@cuichenx can you take a look too since it touches |
Signed-off-by: rghadia <rghadia@nvidia.com>
…avi13/NT3-Ultra-Megatron-Bridge into rghadia/nt3-ultra-gtp-cg Signed-off-by: rghadia <rghadia@nvidia.com>
Contributor
Author
|
/ok to test cbd1d28 |
…into rghadia/nt3-ultra-gtp-cg Signed-off-by: rghadia <rghadia@nvidia.com>
Signed-off-by: rghadia <rghadia@nvidia.com>
Contributor
Author
|
/ok to test 065e181 |
malay-nagda
approved these changes
Oct 13, 2026
malay-nagda
added a commit
that referenced
this pull request
Oct 13, 2026
#6437) Signed-off-by: Ravi Ghadia <rghadia@nvidia.com> Signed-off-by: Malay Nagda <malayn@nvidia.com> Co-authored-by: Ravi Ghadia <40660742+ghadiaravi13@users.noreply.github.com>
2 of 11 tasks
This branch was successfully deployed
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What does this PR do ?
Adds GTP + partial CG support to NT3 Ultra MXFP8 GB200/GB300/VRNVL72 recipes
Also enables cuDNN Layernorm for both GB200 and GB300 recipes.
The GB200/VRNVL72 configs are also updated with NCCLEP support
Changelog
GitHub Actions CI
See the CI section in the Contributing doc for how to trigger the CI. A Nvidia developer will need to approve and trigger the CI for external contributors.
Before your PR is "Ready for review"
Pre checks:
If you haven't finished some of the above items you can still open "Draft" PR.
Additional Information