Skip to content

Setup TorchBench in Docker#158613

Merged
huydhn merged 29 commits intomainfrom
gh/huydhn/6/head
Jul 26, 2025
Merged

Setup TorchBench in Docker#158613
huydhn merged 29 commits intomainfrom
gh/huydhn/6/head

Conversation

@huydhn
Copy link
Copy Markdown
Contributor

@huydhn huydhn commented Jul 18, 2025

This reduces the time spending to setup TorchBench in A100/H100 by another half an hour

Testing

Signed-off-by: Huy Do huydhn@gmail.com

cc @H-Huang @awgu @wanchaol @fegin @fduwjj @wz337 @wconstab @d4l3k @pragupta @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @chenyang78 @kadeng @muchulee8 @amjames @chauhang @aakhundov @coconutruben @Lucaskabela

huydhn added 8 commits July 17, 2025 15:51
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
@huydhn
Copy link
Copy Markdown
Contributor Author

huydhn commented Jul 18, 2025

Stack from ghstack (oldest at bottom):

@huydhn huydhn requested review from a team and jeffdaily as code owners July 18, 2025 00:21
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Jul 18, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit 94fa1bf with merge base 6100ed4 (image):

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

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

huydhn added a commit that referenced this pull request Jul 18, 2025
This reduces the time spending to setup TorchBench in A100/H100 by half
an hour

Signed-off-by: Huy Do <huydhn@gmail.com>
ghstack-source-id: 7425aa2
ghstack-comment-id: 3086043340
Pull-Request: #158613
[ghstack-poisoned]
@huydhn huydhn added the no-runner-experiments Bypass Meta/LF runner determinator label Jul 18, 2025
huydhn added a commit that referenced this pull request Jul 18, 2025
This reduces the time spending to setup TorchBench in A100/H100 by half
an hour

Signed-off-by: Huy Do <huydhn@gmail.com>
ghstack-source-id: 9eaa733
ghstack-comment-id: 3086043340
Pull-Request: #158613
Signed-off-by: Huy Do <huydhn@gmail.com>
huydhn added 3 commits July 17, 2025 18:14
[ghstack-poisoned]
[ghstack-poisoned]
[ghstack-poisoned]
huydhn added a commit that referenced this pull request Jul 18, 2025
This reduces the time spending to setup TorchBench in A100/H100 by half
an hour

Signed-off-by: Huy Do <huydhn@gmail.com>
ghstack-source-id: 536ab13
ghstack-comment-id: 3086043340
Pull-Request: #158613
Signed-off-by: Huy Do <huydhn@gmail.com>
[ghstack-poisoned]
huydhn added a commit that referenced this pull request Jul 18, 2025
This reduces the time spending to setup TorchBench in A100/H100 by
another half an hour

Signed-off-by: Huy Do <huydhn@gmail.com>
ghstack-source-id: c3c6d5b
ghstack-comment-id: 3086043340
Pull-Request: #158613
Signed-off-by: Huy Do <huydhn@gmail.com>
[ghstack-poisoned]
@pytorchmergebot pytorchmergebot added the ci-no-td Do not run TD on this PR label Jul 22, 2025
huydhn added 3 commits July 22, 2025 16:48
Signed-off-by: Huy Do <huydhn@gmail.com>
Signed-off-by: Huy Do <huydhn@gmail.com>
@huydhn huydhn removed ci-no-td Do not run TD on this PR oncall: distributed Add this issue/PR to distributed oncall triage queue labels Jul 26, 2025
@huydhn
Copy link
Copy Markdown
Contributor Author

huydhn commented Jul 26, 2025

@pytorchbot merge

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge failed

Reason: PR 158613 is out of sync with the corresponding revision fa565ea on branch gh/huydhn/6/orig that would be merged into main. This usually happens because there is a non ghstack change in the PR. Please sync them and try again (ex. make the changes on origin/gh/huydhn/6/orig and run ghstack).

Details for Dev Infra team Raised by workflow job

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge failed

Reason: PR 158613 is out of sync with the corresponding revision fa565ea on branch gh/huydhn/6/orig that would be merged into main. This usually happens because there is a non ghstack change in the PR. Please sync them and try again (ex. make the changes on origin/gh/huydhn/6/orig and run ghstack).

Details for Dev Infra team Raised by workflow job

@huydhn
Copy link
Copy Markdown
Contributor Author

huydhn commented Jul 26, 2025

@pytorchbot merge

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge failed

Reason: Found an unexpected number of PRs mentioned in commit 0e37fc8: 0. This is probably because you are using an old version of ghstack. Please update ghstack and resubmit your PRs

Details for Dev Infra team Raised by workflow job

@huydhn
Copy link
Copy Markdown
Contributor Author

huydhn commented Jul 26, 2025

I screwed up my ghstack, but as the PR has been approved and all the signal are ok, I'm going to merge the change

@huydhn huydhn merged commit d72ebef into main Jul 26, 2025
256 of 261 checks passed
@XuehaiPan
Copy link
Copy Markdown
Collaborator

@pytorchbot revert -c nosignal -m "checkout_install_torchbench function is removed but still referenced in trunk"

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

@huydhn your PR has been successfully reverted.

pytorchmergebot added a commit that referenced this pull request Jul 28, 2025
This reverts commit d72ebef.

Reverted #158613 on behalf of https://github.com/XuehaiPan due to checkout_install_torchbench function is removed but still referenced in trunk ([comment](#158613 (comment)))
@pytorchmergebot pytorchmergebot added the ci-no-td Do not run TD on this PR label Jul 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-td Do not run TD on this PR ciflow/inductor ciflow/trunk Trigger trunk jobs on your pull request Merged no-runner-experiments Bypass Meta/LF runner determinator release notes: releng release notes category Reverted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants