FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Fix/fix autotp universal checkpoint ci by tohtana · Pull Request #7937 · deepspeedai/DeepSpeed · GitHub

Fix/fix autotp universal checkpoint ci - #7937

Merged
sfc-gh-truwase merged 2 commits into
deepspeedai:masterfrom
tohtana:fix/fix-autotp-universal-checkpoint-ci
Mar 31, 2026
Merged

Fix/fix autotp universal checkpoint ci#7937
sfc-gh-truwase merged 2 commits into
deepspeedai:masterfrom
tohtana:fix/fix-autotp-universal-checkpoint-ci

Conversation

tohtana commented Mar 31, 2026

Copy link
Copy Markdown
Collaborator

The full CI test fails throwing "RuntimeError: Cannot re-initialize CUDA" because of tests for universal checkpoint and AutoTP.

It happens because they run torch.cuda.current_device() under pytest --forked. As the tests only touch universal checkpoint metadata, we won't need to call it. This PR skips constructor-time AutoTP materialization when mp_group is None.
Partitioning still happens in the real AutoTP usage where an actual model-parallel group is given.

tohtana added 2 commits March 30, 2026 13:24
Signed-off-by: Masahiro Tanaka <mtanaka@anyscale.com>
Signed-off-by: Masahiro Tanaka <mtanaka@anyscale.com>
sfc-gh-truwase merged commit 3bdebc0 into deepspeedai:master Mar 31, 2026
10 of 13 checks passed
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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL