Skip to content

Fix torch.compile smoke test include it for py 3.12, 3.13#8876

Merged
atalman merged 2 commits intopytorch:mainfrom
atalman:enable_smoke_test_compile
Jan 24, 2025
Merged

Fix torch.compile smoke test include it for py 3.12, 3.13#8876
atalman merged 2 commits intopytorch:mainfrom
atalman:enable_smoke_test_compile

Conversation

@atalman
Copy link
Copy Markdown
Contributor

@atalman atalman commented Jan 23, 2025 •
edited
Loading

Run toch compile test on py 3.12 and 3.13 however exclude 3.13t

Test:

>>> is_gil = not sysconfig.get_config_var("Py_GIL_DISABLED")
>>> print(f"{is_gil}")
True
>>> print(sysconfig.get_config_var("Py_GIL_DISABLED"))
None
>>>

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot bot commented Jan 23, 2025 •
edited
Loading

🔗 Helpful Links

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

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

❌ 1 New Failure, 19 Pending

As of commit a3263c1 with merge base 4315f8a (image):

NEW FAILURE - The following job has failed:

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

@atalman atalman changed the title Fix vision torch.compile test Fix torch.compile smoke test include it for py 3.12, 3.13 Jan 23, 2025
kit1980
kit1980 suggested changes Jan 23, 2025
View reviewed changes
@atalman atalman merged commit 867521e into pytorch:main Jan 24, 2025
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 24, 2025

Hey @atalman!

You merged this PR, but no labels were added.
The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

facebook-github-bot pushed a commit that referenced this pull request Jul 9, 2025
@NicolasHug @facebook-github-bot
[fbsync] Fix torch.compile smoke test include it for py 3.12, 3.13 (#…
686c9fd 
…8876)

Reviewed By: scotts

Differential Revision: D77997060

fbshipit-source-id: 808960bc7424a342b5954f3b49c3f1110fa4fd07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@seemethere seemethere seemethere approved these changes

@malfet malfet malfet approved these changes

+1 more reviewer

@kit1980 kit1980 kit1980 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cla signed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@atalman @kit1980 @seemethere @malfet @facebook-github-bot