Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use compiler.is_compiling() instead of deprecated _dynamo.is_compiling() #8455

Merged
merged 2 commits into from
May 30, 2024

Conversation

NicolasHug
Copy link
Member

@NicolasHug NicolasHug commented May 30, 2024

v2 transforms tests are failing because of the deprecation warning (treated as error)

2024-05-30T09:08:14.0827202Z FAILED test/test_transforms_v2.py::TestScaleJitter::test_transform[cpu-make_video] - FutureWarning: `is_compiling` is deprecated. Use `torch.compiler.is_compiling()` instead.

https://github.com/pytorch/vision/actions/runs/9299571569/job/25593772937?pr=7990

cc @vfdev-5

Copy link

pytorch-bot bot commented May 30, 2024

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

❌ 17 New Failures, 3 Unrelated Failures

As of commit ffe23b0 with merge base 9b4b742 (image):

NEW FAILURES - The following jobs have failed:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

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

Copy link
Collaborator

@vfdev-5 vfdev-5 left a comment

Choose a reason for hiding this comment

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

LGTM

@NicolasHug NicolasHug merged commit b0f9f7b into pytorch:main May 30, 2024
47 of 69 checks passed
@NicolasHug NicolasHug deleted the laefjnljaenf branch May 30, 2024 09:36
facebook-github-bot pushed a commit that referenced this pull request Jun 7, 2024
…_compiling() (#8455)

Reviewed By: vmoens

Differential Revision: D58283846

fbshipit-source-id: 5b4bc865fdfc664f44788c6e186ec1bffd05dec8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants