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

Introducing ORTPipelineModule - DeepSpeed Parallel Pipeline Support. #20287

Merged
merged 9 commits into from
Apr 18, 2024

Conversation

AdamLouly
Copy link
Contributor

@AdamLouly AdamLouly commented Apr 11, 2024

Description

Introducing a new class ORTPipelineModule to handle wrapping layers in DeepSpeed pipeline parallel.

Motivation and Context

To support pipeline parallelism on ORTModule.

This PR will include an initial support of deepspeed Pipeline parallelism.

  • Support Pipeline parallel where layers are nn Modules in Sequential.
  • Support LayerSpec and TiedLayerSpec
  • Enable partitioning to accept List
  • Full-GPU Graph Consolidation
  • Subgraph Merging for Inference

@AdamLouly AdamLouly requested review from wschin and pengwa April 11, 2024 21:35
wschin
wschin previously approved these changes Apr 17, 2024
@AdamLouly
Copy link
Contributor Author

Thanks for the review

@AdamLouly AdamLouly merged commit ee74fb6 into main Apr 18, 2024
92 of 94 checks passed
@AdamLouly AdamLouly deleted the adamlouly/ds_pipe branch April 18, 2024 18:30
TedThemistokleous pushed a commit to TedThemistokleous/onnxruntime that referenced this pull request May 7, 2024
…icrosoft#20287)

### Description
Introducing a new class ORTPipelineModule to handle wrapping layers in
DeepSpeed pipeline parallel.


### Motivation and Context
To support pipeline parallelism on ORTModule.

This PR will include an initial support of deepspeed Pipeline
parallelism.

- [x] Support Pipeline parallel where layers are nn Modules in
Sequential.
- [ ] Support LayerSpec and TiedLayerSpec
- [ ] Enable partitioning to accept List
- [ ] Full-GPU Graph Consolidation
- [ ] Subgraph Merging for Inference
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.

2 participants