Skip to content

[Windows] Add model run CI - #13836

Merged
GregoryComer merged 98 commits into
mainfrom
gh/GregoryComer/150/head
Sep 3, 2025
Merged

[Windows] Add model run CI#13836
GregoryComer merged 98 commits into
mainfrom
gh/GregoryComer/150/head

Conversation

@GregoryComer

@GregoryComerGregoryComer commented Aug 31, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

Add Windows CI jobs to test models. From looking at the existing CI, it looks like our model test jobs are pretty minimal and don't verify much other than it runs without crashing. I'd like to re-visit this in the near future and integrate with the new test harness, but for the purpose this PR, I'm just matching the Windows jobs to what the Linux jobs do.

GregoryComerand others added 30 commits August 26, 2025 15:48
@GregoryComer
Update
d6a70d6
[ghstack-poisoned]
@GregoryComer
Update
18866f4
[ghstack-poisoned]
@GregoryComer
Update
e7423a4
[ghstack-poisoned]
@GregoryComer
Update
323d96c
[ghstack-poisoned]
@GregoryComer
Update
bca23f5
[ghstack-poisoned]
@GregoryComer
Update
ab69e35
[ghstack-poisoned]
@GregoryComer
Update
c47d467
[ghstack-poisoned]
@GregoryComer
Update
cd8e9bc
[ghstack-poisoned]
@GregoryComer
Update
cb60871
[ghstack-poisoned]
@GregoryComer
Update
19dda19
[ghstack-poisoned]
@GregoryComer
Update
cd0178f
[ghstack-poisoned]
@GregoryComer
Update
ed4cffd
[ghstack-poisoned]
@GregoryComer
Update
934682f
[ghstack-poisoned]
@GregoryComer
Update
ef62839
[ghstack-poisoned]
@GregoryComer
Update
441fd5f
[ghstack-poisoned]
@GregoryComer
Update
7010cf3
[ghstack-poisoned]
@GregoryComer
Update
3cfae60
[ghstack-poisoned]
@GregoryComer
Update
b583635
[ghstack-poisoned]
@GregoryComer
Update
ed133f7
[ghstack-poisoned]
@GregoryComer
Update
228e146
[ghstack-poisoned]
@GregoryComer
Update
b64bd0f
[ghstack-poisoned]
@GregoryComer
Update
fe40834
[ghstack-poisoned]
@GregoryComer
Update
b97ba63
[ghstack-poisoned]
@GregoryComer
Update
5be90d9
[ghstack-poisoned]
@GregoryComer
Update
f15e674
[ghstack-poisoned]
@GregoryComer
Update
33da5e3
[ghstack-poisoned]
@GregoryComer
Update
2a6f87b
[ghstack-poisoned]
@GregoryComer
Update
8ce9df7
[ghstack-poisoned]
@GregoryComer
Update
d5efbf9
[ghstack-poisoned]
@GregoryComer
Update
e918a78
[ghstack-poisoned]
@GregoryComer
Update
fdd8e12
[ghstack-poisoned]
@GregoryComer
Update
c285ecf
[ghstack-poisoned]
GregoryComerand others added 10 commits September 2, 2025 17:59
@GregoryComer
Update
edfd321
[ghstack-poisoned]
@GregoryComer
Update
95856ad
[ghstack-poisoned]
@GregoryComer
Update
b41ddee
[ghstack-poisoned]
@GregoryComer
Update
15b9ce0
[ghstack-poisoned]
@GregoryComer
Update
fd3d9c3
[ghstack-poisoned]
@GregoryComer
Update
fc9d9b9
[ghstack-poisoned]
@GregoryComer
Update
8a4a0e3
[ghstack-poisoned]
@GregoryComer
Update
aae2ad8
[ghstack-poisoned]
@GregoryComer
Update
fbde97d
[ghstack-poisoned]
@GregoryComer
Update
056b85d
[ghstack-poisoned]
@GregoryComer
GregoryComer marked this pull request as ready for review September 3, 2025 06:06
mergennachin
mergennachin approved these changes Sep 3, 2025
View reviewed changes
Comment thread.ci/scripts/test_model.ps1 Outdated
Comment thread.ci/scripts/test_model.ps1 Outdated
Base automatically changed from gh/GregoryComer/146/head to mainSeptember 3, 2025 19:06
@GregoryComer
Update
0360486
[ghstack-poisoned]
@github-actions

github-actionsBot commented Sep 3, 2025

Copy link
Copy Markdown

This PR needs a release notes: label

If your change should be included in the release notes (i.e. would users of this library care about this change?), please use a label starting with release notes:. This helps us keep track and include your important work in the next release notes.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "release notes: none"

For more information, see
https://github.com/pytorch/pytorch/wiki/PyTorch-AutoLabel-Bot#why-categorize-for-release-notes-and-how-does-it-work.

@GregoryComer
Update
d521e08
[ghstack-poisoned]
@GregoryComer
GregoryComer merged commit 66799e3 into mainSep 3, 2025
285 of 286 checks passed
@GregoryComer
GregoryComer deleted the gh/GregoryComer/150/head branch September 3, 2025 20:57
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mergennachinmergennachinmergennachin approved these changes
@JacobSzwejbkaJacobSzwejbkaAwaiting requested review from JacobSzwejbka
@larryliu0820larryliu0820Awaiting requested review from larryliu0820
@kirklandsignkirklandsignAwaiting requested review from kirklandsign
@manuelcandalesmanuelcandalesAwaiting requested review from manuelcandales
@swolchokswolchokAwaiting requested review from swolchok
@jackzhxngjackzhxngAwaiting requested review from jackzhxng
@lucylqlucylqAwaiting requested review from lucylq
@cccclaicccclaiAwaiting requested review from cccclai
@metascroymetascroyAwaiting requested review from metascroy

Assignees

No one assigned

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@GregoryComer@mergennachin