Skip to content

Update distributed training for MetaTensor #777

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

Merged
merged 6 commits into from
Jul 11, 2022
Merged

Conversation

Nic-Ma
Copy link
Contributor

@Nic-Ma Nic-Ma commented Jul 11, 2022

Description

This PR updated all the distributed training tutorials to adapt to the MetaTensor.

Status

Ready

Checks

  • Notebook runs automatically ./runner [-p <regex_pattern>]

@Nic-Ma Nic-Ma force-pushed the update-dist-tutorials branch from b499100 to 9441bf0 Compare July 11, 2022 10:40
@Nic-Ma Nic-Ma changed the title [WIP] Update distributed training for MetaTensor Update distributed training for MetaTensor Jul 11, 2022
@Nic-Ma Nic-Ma marked this pull request as ready for review July 11, 2022 10:42
@Nic-Ma
Copy link
Contributor Author

Nic-Ma commented Jul 11, 2022

Hi @wyli , @rijobro ,

I have updated and verified all the distributed training / evaluation examples on 2 V100 GPUs.
Could you please help review it?

Thanks in advance.

@wyli wyli merged commit b105849 into main Jul 11, 2022
@wyli wyli deleted the update-dist-tutorials branch July 11, 2022 12:05
@wyli
Copy link
Contributor

wyli commented Jul 11, 2022

thanks, could you pls also help review this related https://github.com/Project-MONAI/tutorials/blob/main/modules/compute_metric.py @Nic-Ma

@Nic-Ma
Copy link
Contributor Author

Nic-Ma commented Jul 11, 2022

thanks, could you pls also help review this related https://github.com/Project-MONAI/tutorials/blob/main/modules/compute_metric.py @Nic-Ma

Sure, I will update it and test soon.

Thanks.

@Nic-Ma
Copy link
Contributor Author

Nic-Ma commented Jul 12, 2022

Submitted PR for it:#780

Thanks.

boneseva pushed a commit to boneseva/MONAI-tutorials that referenced this pull request Apr 21, 2024
* [DLMED] update brats ddp

Signed-off-by: Nic Ma <[email protected]>

* [DLMED] restore dist for compatible

Signed-off-by: Nic Ma <[email protected]>

* [DLMED] update ddp examples

Signed-off-by: Nic Ma <[email protected]>

* [DLMED] update ignite workflows

Signed-off-by: Nic Ma <[email protected]>

* [DLMED] update smart cache ddp

Signed-off-by: Nic Ma <[email protected]>

* [DLMED] update horovod ddp

Signed-off-by: Nic Ma <[email protected]>
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