KEMBAR78
[Dynamo] stop import third-party astunparse by zou3519 · Pull Request #142503 · pytorch/pytorch · GitHub
Skip to content

Conversation

@zou3519
Copy link
Contributor

@zou3519 zou3519 commented Dec 10, 2024

PyTorch's minimum version is 3.9, so we can now use ast.unparse.

Test Plan:
- wait for tests

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Dec 10, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 4abcf01 with merge base 5d3bc63 (image):
💚 Looks good so far! There are no failures yet. 💚

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

zou3519 added a commit that referenced this pull request Dec 10, 2024
PyTorch's minimum version is 3.9, so we can now use ast.unparse.

Test Plan:
- wait for tests

ghstack-source-id: 9f90461
Pull Request resolved: #142503
@zou3519 zou3519 added the topic: not user facing topic category label Dec 10, 2024
Copy link
Contributor

@StrongerXi StrongerXi left a comment

Choose a reason for hiding this comment

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

Sweet

pytorchmergebot pushed a commit that referenced this pull request Dec 11, 2024
Fixes #127738
Fixes #127755

In the discussion in #127738 we
determined that this is not fixable, so we're just going to skip the
test.

Test Plan:
- tests
Pull Request resolved: #142508
Approved by: https://github.com/StrongerXi, https://github.com/yanboliang, https://github.com/mlazos
ghstack dependencies: #142502, #142503
@github-actions github-actions bot deleted the gh/zou3519/1101/head branch January 11, 2025 02:10
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.

5 participants