Skip to content

Build Windows binaries with Visual Studio 2022 Build Tools#90855

Closed
Blackhex wants to merge 1 commit intopytorch:masterfrom
Blackhex:vs-2022-built-binaries
Closed

Build Windows binaries with Visual Studio 2022 Build Tools#90855
Blackhex wants to merge 1 commit intopytorch:masterfrom
Blackhex:vs-2022-built-binaries

Conversation

@Blackhex
Copy link
Copy Markdown
Collaborator

@Blackhex Blackhex commented Dec 14, 2022

This PR enables VS 2022 binaries for build and test jobs. Another PR pytorch/builder#1240 is doing majority of the work.

Closes #87695.

cc @peterjc123 @mszhanyi @skyline75489 @nbcsm @seemethere @malfet @pytorch/pytorch-dev-infra

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Dec 14, 2022
@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Dec 14, 2022

🔗 Helpful Links

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

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

⏳ No Failures, 3 Pending

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

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

@Blackhex Blackhex added the ciflow/binaries Trigger all binary build and upload jobs on the PR label Dec 14, 2022
@Blackhex Blackhex force-pushed the vs-2022-built-binaries branch 4 times, most recently from 2c11621 to e1e807d Compare December 16, 2022 10:08
@Blackhex Blackhex changed the title [WIP] Use Blackhex/pytorch-builder for testing purposes. [WIP] Build Windows binaries with Visual Studio 2022 Build Tools. Dec 16, 2022
@Blackhex Blackhex requested a review from jeanschmidt December 16, 2022 14:47
@Blackhex Blackhex added module: windows Windows support for PyTorch module: ci Related to continuous integration and removed topic: not user facing topic category labels Dec 19, 2022
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Dec 19, 2022
@Blackhex Blackhex force-pushed the vs-2022-built-binaries branch 4 times, most recently from 8e18f2e to f4ee44f Compare December 20, 2022 09:34
@Blackhex Blackhex force-pushed the vs-2022-built-binaries branch 3 times, most recently from 9e20a01 to 087e290 Compare January 5, 2023 11:29
@Blackhex Blackhex changed the title [WIP] Build Windows binaries with Visual Studio 2022 Build Tools. [WIP] Build Windows binaries with Visual Studio 2022 Build Tools Jan 5, 2023
@Blackhex Blackhex force-pushed the vs-2022-built-binaries branch 7 times, most recently from 0116b0d to 16b26bc Compare January 7, 2023 18:23
@Blackhex Blackhex force-pushed the vs-2022-built-binaries branch from 16b26bc to 8d93d3b Compare January 27, 2023 10:53
@Blackhex Blackhex changed the title [WIP] Build Windows binaries with Visual Studio 2022 Build Tools Build Windows binaries with Visual Studio 2022 Build Tools Jan 27, 2023
@Blackhex Blackhex self-assigned this Jan 27, 2023
@Blackhex Blackhex marked this pull request as ready for review January 27, 2023 22:51
@Blackhex Blackhex requested review from atalman and malfet January 27, 2023 22:52
@ngimel ngimel added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jan 28, 2023
@jeanschmidt
Copy link
Copy Markdown
Contributor

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Feb 7, 2023
@pytorchmergebot
Copy link
Copy Markdown
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

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Merge failed

Reason: This PR is too stale; the last push date was more than 3 days ago. Please rebase and try again. You can rebase by leaving the following comment on this PR:
@pytorchbot rebase

Details for Dev Infra team Raised by workflow job

@jeanschmidt
Copy link
Copy Markdown
Contributor

@pytorchbot rebase

@Blackhex Blackhex force-pushed the vs-2022-built-binaries branch from 8d93d3b to c758715 Compare February 7, 2023 08:44
@pytorchmergebot
Copy link
Copy Markdown
Collaborator

@pytorchbot successfully started a rebase job. Check the current status here

@pytorchmergebot
Copy link
Copy Markdown
Collaborator

Tried to rebase and push PR #90855, but it was already up to date

@seemethere
Copy link
Copy Markdown
Member

@pytorchbot merge

@pytorchmergebot
Copy link
Copy Markdown
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

@Blackhex Blackhex deleted the vs-2022-built-binaries branch February 8, 2023 08:00
huydhn added a commit to huydhn/pytorch that referenced this pull request Apr 20, 2023
pytorchmergebot pushed a commit that referenced this pull request Apr 20, 2023
…90855) (#99591)

This reverts commit a88c15a.  Once we have the AMI ready, we can revert this and use VS2022 again.  This is to mitigate flaky Windows build in trunk pytorch/builder#1387.

Note that as VS2019 is already available in the current AMI, it won't be installed again per logic in https://github.com/pytorch/builder/blob/main/windows/internal/vs2019_install.ps1#L25-L29. Thus, this helps avoid the flaky installation issue.
Pull Request resolved: #99591
Approved by: https://github.com/kit1980, https://github.com/Blackhex, https://github.com/malfet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries Trigger all binary build and upload jobs on the PR ciflow/trunk Trigger trunk jobs on your pull request Merged module: ci Related to continuous integration module: windows Windows support for PyTorch open source topic: not user facing topic category triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for VS2022 build chain

6 participants