Skip to content
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

add option to disable pipeline partitioning #4322

Merged
merged 8 commits into from Nov 8, 2023

Conversation

nelyahu
Copy link
Contributor

@nelyahu nelyahu commented Sep 13, 2023

added pipeline configuration to deepspeed config under pipeline section pipe_partitioned = "auto"
grad_partitioned = "auto"
can be used to enable or disable activations and grads partitioning in TP/PP mode

@nelyahu
Copy link
Contributor Author

nelyahu commented Sep 19, 2023

@ShadenSmith @jeffra @duli2012 @tjruwase - can you please review?

@nelyahu
Copy link
Contributor Author

nelyahu commented Oct 4, 2023

@tjruwase Hi :) can someone help reviewing this change?

@nelyahu nelyahu force-pushed the disable_pipeline_partitioning branch from c5f497e to 85ee4ac Compare October 9, 2023 06:56
added pipeline configuration to deepspeed config under pipeline section,
with defaults of
pipe_partitioned = True
grad_partitioned = True
can be used to enable or disable activations and grads partitioning in TP/PP mode.
When not using TP, activation and grads partitioning will be disabled
regardless of this confgiuration.
@nelyahu nelyahu force-pushed the disable_pipeline_partitioning branch from 85ee4ac to 0885438 Compare October 9, 2023 06:58
@loadams
Copy link
Contributor

loadams commented Nov 7, 2023

@tjruwase - should we add this to any docs we have?

@tjruwase
Copy link
Contributor

tjruwase commented Nov 8, 2023

@tjruwase - should we add this to any docs we have?

@loadams, yes, we should. But I don't believe we have any docs for DeepSpeed pipeline parallelism support.

@loadams loadams added this pull request to the merge queue Nov 8, 2023
Merged via the queue into microsoft:master with commit c2074b3 Nov 8, 2023
15 checks passed
@nelyahu nelyahu deleted the disable_pipeline_partitioning branch December 12, 2023 14:44
mauryaavinash95 pushed a commit to mauryaavinash95/DeepSpeed that referenced this pull request Feb 17, 2024
added pipeline configuration to deepspeed config under pipeline section
pipe_partitioned = "auto"
grad_partitioned = "auto"
can be used to enable or disable activations and grads partitioning in
TP/PP mode

Co-authored-by: Olatunji Ruwase <olruwase@microsoft.com>
Co-authored-by: Logan Adams <114770087+loadams@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants