Skip to content

[FSDP][Easy] Fix context manager syntax#91410

Closed
awgu wants to merge 1 commit intogh/awgu/287/basefrom
gh/awgu/287/head
Closed

[FSDP][Easy] Fix context manager syntax#91410
awgu wants to merge 1 commit intogh/awgu/287/basefrom
gh/awgu/287/head

Conversation

@awgu
Copy link
Collaborator

@awgu awgu commented Dec 27, 2022

Stack from ghstack:

@pytorch-bot
Copy link

pytorch-bot bot commented Dec 27, 2022

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/91410

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 112794c:
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

awgu pushed a commit that referenced this pull request Dec 27, 2022
ghstack-source-id: 647101b
Pull Request resolved: #91410
@awgu awgu added release notes: distributed (fsdp) release notes category topic: not user facing topic category and removed release notes: distributed (sharded) release notes category labels Dec 27, 2022
@kit1980
Copy link
Contributor

kit1980 commented Dec 27, 2022

Let's wait with merging this until we understand what's going on with #91371 so we don't mask the symptoms.

@kit1980
Copy link
Contributor

kit1980 commented Dec 28, 2022

@pytorchbot merge -f "Unrelated ios failure"

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@facebook-github-bot facebook-github-bot deleted the gh/awgu/287/head branch June 8, 2023 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants