8000 [dynamo, nested graph breaks] add nested graph break tests by williamwen42 · Pull Request #144516 · pytorch/pytorch · GitHub

[dynamo, nested graph breaks] add nested graph break tests #144516

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.

Sign up for GitHub

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

Jump to bottom

Draft

williamwen42 wants to merge 10 commits into gh/williamwen42/196/base from gh/williamwen42/196/head

+534 −11

Member

williamwen42 commented

•

Stack from ghstack (oldest at bottom):

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @chauhang @amjames


          add nested graph break tests

40e526d

[ghstack-poisoned]

pytorch-bot bot commented

•

🔗 Helpful Links

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

📄 Preview Python docs built from this PR
📄 Preview C++ docs built from this PR
❓ Need help or want to give feedback on the CI? Visit the bot commands wiki or our office hours

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:

CUDA not found in NVIDIA runners

❌ 7 New Failures

As of commit a2d97b2 with merge base c2dda47 ():

NEW FAILURES - The following jobs have failed:

Check Labels / Check labels (gh)
RuntimeError: Error checking labels: PR does not have required labels
pull / linux-focal-cuda12.6-py3.10-gcc11 / test (default, 1, 5, linux.4xlarge.nvidia.gpu) (gh)
dynamo/test_nested_graph_breaks.py::NestedGraphBreakTests::test_cells
pull / linux-focal-cuda12.6-py3.10-gcc11-sm89 / test (default, 1, 5, linux.g6.4xlarge.experimental.nvidia.gpu) (gh)
dynamo/test_nested_graph_breaks.py::NestedGraphBreakTests::test_cells
pull / linux-focal-py3.13-clang10 / test (default, 1, 5, linux.4xlarge) (gh)
dynamo/test_nested_graph_breaks.py::NestedGraphBreakTests::test_cells
pull / linux-focal-py3.9-clang10 / test (default, 1, 5, linux.4xlarge) (gh)
dynamo/test_nested_graph_breaks.py::NestedGraphBreakTests::test_cells
pull / linux-jammy-py3.10-clang15-asan / test (default, 1, 6, linux.4xlarge) (gh)
dynamo/test_nested_graph_breaks.py::NestedGraphBreakTests::test_cells
pull / linux-jammy-py3.9-gcc11 / test (default, 1, 5, linux.2xlarge) (gh)
dynamo/test_nested_graph_breaks.py::NestedGraphBreakTests::test_cells

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

pytorch-bot bot added ciflow/inductor module: dynamo labels

williamwen42 added a commit that referenced this pull request


          add nested graph break tests

cb31079

ghstack-source-id: 2e25912
Pull Request resolved: #144516

Contributor

github-actions bot commented

This PR needs a `release notes:` label

If your changes are user facing and intended to be a part of release notes, please use a label starting with release notes:.

If not, please add the topic: not user facing label.

To add a label, you can comment to pytorchbot, for example
@pytorchbot label "topic: not user facing"

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

williamwen42 marked this pull request as draft

January 10, 2025 00:16

Contributor

github-actions bot commented

Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as Stale.
Feel free to remove the Stale label if you feel this was a mistake.
If you are unable to remove the Stale label please contact a maintainer in order to do so.
If you want the bot to never mark this PR stale again, add the no-stale label.
Stale pull requests will automatically be closed after 30 days of inactivity.

github-actions bot added the Stale label

williamwen42 removed the Stale label


          Update on "add nested graph break tests"

04e409b

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

williamwen42 mentioned this pull request

[dynamo, nested graph breaks] small fixes to resume function generation #151056

Draft

williamwen42 changed the title ~~add nested graph break tests~~ [dynamo, nested graph breaks] add nested graph break tests


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

0415dc9

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

Divigroup-RAP pushed a commit to Divigroup-RAP/PYTORCH that referenced this pull request


          add nested graph break tests

3ceaebf

ghstack-source-id: eef6717
Pull Request resolved: pytorch/pytorch#144516


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

59b736c

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

williamwen42 mentioned this pull request

[dynamo, nested graph breaks] refactor codegen to minimize NULL codegen'ing #153510

Draft


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

01b5875

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

williamwen42 added a commit that referenced this pull request


          add nested graph break tests

86005d0

ghstack-source-id: f0a3375
Pull Request resolved: #144516


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

09c01a2

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

williamwen42 added a commit that referenced this pull request


          add nested graph break tests

05a0881

ghstack-source-id: 9935482
Pull Request resolved: #144516


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

b1ab7a0

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

williamwen42 added a commit that referenced this pull request


          add nested graph break tests

fed78e7

ghstack-source-id: 8c9ecfa
Pull Request resolved: #144516


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

5a419d0

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

williamwen42 added a commit that referenced this pull request


          add nested graph break tests

c31d0e0

ghstack-source-id: f415878
Pull Request resolved: #144516


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

a3f4af1

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

williamwen42 added a commit that referenced this pull request


          add nested graph break tests

d825618

ghstack-source-id: 429d375
Pull Request resolved: #144516


          Update on "[dynamo, nested graph breaks] add nested graph break tests"

a2d97b2

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx chenyang78 kadeng chauhang amjames

[ghstack-poisoned]

This was referenced May 17, 2025

[dynamo, nested graph breaks] remove block stack graph break in output_graph #153772

Draft

[dynamo, nested graph breaks] add skip_frame debugging function #153773

Draft

williamwen42 added a commit that referenced this pull request


          add nested graph break tests

d6dc4bb

ghstack-source-id: 2ba774e
Pull Request resolved: #144516

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/inductor module: dynamo

0