Repository navigation
fix(slurm): default pipeline parallelism to one - #1116
Conversation
Signed-off-by: Nabin Mulepati <nmulepati@nvidia.com>
|
Fern preview: https://nvidia-preview-pr-1116.docs.buildwithfern.com/nemo/datadesigner
|
[Medium impact] The PR appears safe to merge within its stated saved-run compatibility limits. SummaryThis PR defaults
Reviews (3) · Last reviewed commit: "fix(slurm): use a compatible benchmark b..." · Reviewed by Greptile |
Signed-off-by: Nabin Mulepati <nmulepati@nvidia.com>
andreatnvidia
left a comment
There was a problem hiding this comment.
Please address the benchmark example issue before merging.
Signed-off-by: Nabin Mulepati <nmulepati@nvidia.com>
📋 Summary
Default pipeline parallelism to one stage, independently of the number of nodes per replica. A TP16 deployment across two eight-GPU nodes now works with PP omitted; additional pipeline stages must be requested explicitly. Expert parallelism remains disabled by default and opt-in for compatible MoE models.
Earlier persisted runs are not migrated; compatibility with their prior topology and serialization semantics is intentionally outside this change.
🔗 Related Issue
Related to #850; follows #1112.
🔄 Changes
pipeline_parallel: 1in authored configuration and topology derivation, removing the node-count fallback.🧪 Testing
✅ Checklist