Skip to content

Conversation

@XilunWu
Copy link
Contributor

@XilunWu XilunWu commented Apr 25, 2023

Why

This PR adds test_device_mesh.py to periodic tests because it requires more GPUs (4/8) to run testing than the CI machine has.

Stack from ghstack (oldest at bottom):

@pytorch-bot
Copy link

pytorch-bot bot commented Apr 25, 2023

🔗 Helpful Links

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

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

✅ No Failures

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

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

XilunWu added a commit that referenced this pull request Apr 25, 2023
@XilunWu XilunWu requested review from fduwjj and wanchaol April 25, 2023 21:31
@XilunWu XilunWu added topic: not user facing topic category ciflow/periodic Trigger jobs ran periodically on master (periodic.yml) on the PR labels Apr 25, 2023
@XilunWu
Copy link
Contributor Author

XilunWu commented Apr 26, 2023

@pytorchmergebot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Apr 26, 2023
@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

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

Copy link
Contributor

@fduwjj fduwjj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@XilunWu XilunWu deleted the gh/XilunWu/28/head branch April 27, 2023 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/periodic Trigger jobs ran periodically on master (periodic.yml) on the PR ciflow/trunk Trigger trunk jobs on your pull request Merged merging topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants