Hoist set_feature_use out of conditional, rename some variables #141683

ezyang · 2024-11-27T13:49:05Z

Stack from ghstack (oldest at bottom):

Signed-off-by: Edward Z. Yang ezyang@meta.com

cc @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @ColinPeppler @amjames @desertfire @chauhang @aakhundov

[ghstack-poisoned]

pytorch-bot · 2024-11-27T13:49:09Z

🔗 Helpful Links

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

📄 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.

✅ You can merge normally! (2 Unrelated Failures)

As of commit 6f35720 with merge base 0f261e8 ():

UNSTABLE - The following jobs failed but were likely due to flakiness present on trunk and has been marked as unstable:

inductor / cuda12.1-py3.10-gcc9-sm86 / test (inductor_timm, 1, 2, linux.g5.4xlarge.nvidia.gpu, unstable) (gh) (#141703)
convnext_base
inductor / cuda12.4-py3.10-gcc9-sm86 / test (inductor_timm, 1, 2, linux.g5.4xlarge.nvidia.gpu, unstable) (gh) (#141498)
convnext_base

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

Signed-off-by: Edward Z. Yang <ezyang@meta.com> ghstack-source-id: 9afb56f Pull Request resolved: #141683

[ghstack-poisoned]

ezyang · 2024-11-27T20:56:36Z

@pytorchbot merge

ezyang · 2024-11-28T04:44:16Z

@pytorchbot merge -f "unrelated failures"

pytorchmergebot · 2024-11-28T04:46:00Z

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging

Check the merge workflow status
here

pytorchmergebot · 2024-11-28T04:46:26Z

Merge failed

Reason: Command git -C /home/runner/work/pytorch/pytorch cherry-pick -x 1e6544ef902eb7b3d937020388d6a7f0f73228b6 returned non-zero exit code 1

Auto-merging torch/_inductor/compile_fx.py
CONFLICT (content): Merge conflict in torch/_inductor/compile_fx.py
error: could not apply 1e6544ef902... Inline FxGraphCache.load into its sole call site
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git cherry-pick --continue".
hint: You can instead skip this commit with "git cherry-pick --skip".
hint: To abort and get back to the state before "git cherry-pick",
hint: run "git cherry-pick --abort".
hint: Disable this message with "git config advice.mergeConflict false"

Details for Dev Infra team

Raised by workflow job

[ghstack-poisoned]

ezyang · 2024-11-28T14:19:29Z

@pytorchbot merge

pytorchmergebot · 2024-11-28T14:22:02Z

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging

Check the merge workflow status
here

I was constantly annoyed at the fact that we had a separate else branch for when cache was disabled which was distinct from when cache was bypassed. This diff gets rid of the disabled cache branch, so we use the same logic for bypass/disable. I actually think this change probably didn't actually matter much for the POC but I think it's cleaner. Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: #141685 Approved by: https://github.com/aorenste ghstack dependencies: #141681, #141683

Stacked on top of #141685 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: #141688 Approved by: https://github.com/Skylion007, https://github.com/jansel ghstack dependencies: #141681, #141683, #141685

Stacked on #141688 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: #141689 Approved by: https://github.com/jansel ghstack dependencies: #141681, #141683, #141685, #141688

Stacked on #141689 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: #141691 Approved by: https://github.com/jansel ghstack dependencies: #141681, #141683, #141685, #141688, #141689

Stacked on #141691 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: #141695 Approved by: https://github.com/aorenste ghstack dependencies: #141681, #141683, #141685, #141688, #141689, #141691

…rch#141683) Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: pytorch#141683 Approved by: https://github.com/jamesjwu, https://github.com/jansel ghstack dependencies: pytorch#141681

I was constantly annoyed at the fact that we had a separate else branch for when cache was disabled which was distinct from when cache was bypassed. This diff gets rid of the disabled cache branch, so we use the same logic for bypass/disable. I actually think this change probably didn't actually matter much for the POC but I think it's cleaner. Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: pytorch#141685 Approved by: https://github.com/aorenste ghstack dependencies: pytorch#141681, pytorch#141683

Stacked on top of pytorch#141685 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: pytorch#141688 Approved by: https://github.com/Skylion007, https://github.com/jansel ghstack dependencies: pytorch#141681, pytorch#141683, pytorch#141685

Stacked on pytorch#141688 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: pytorch#141689 Approved by: https://github.com/jansel ghstack dependencies: pytorch#141681, pytorch#141683, pytorch#141685, pytorch#141688

Stacked on pytorch#141689 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: pytorch#141691 Approved by: https://github.com/jansel ghstack dependencies: pytorch#141681, pytorch#141683, pytorch#141685, pytorch#141688, pytorch#141689

Stacked on pytorch#141691 Signed-off-by: Edward Z. Yang <ezyang@meta.com> Pull Request resolved: pytorch#141695 Approved by: https://github.com/aorenste ghstack dependencies: pytorch#141681, pytorch#141683, pytorch#141685, pytorch#141688, pytorch#141689, pytorch#141691

Signed-off-by: Edward Z. Yang <ezyang@meta.com> ghstack-source-id: aa3537e Pull Request resolved: pytorch/pytorch#141683

Update

cc0b571

[ghstack-poisoned]

This was referenced Nov 27, 2024

Code motion CompiledFxGraph to a dedicated file #141654

Closed

Move post compile steps into post_compile1/post_compile2 method #141656

Closed

ezyang mentioned this pull request Nov 27, 2024

Inline FxGraphCache.load into its sole call site #141681

Closed

pytorch-bot bot added ciflow/inductor module: inductor labels Nov 27, 2024

github-actions bot requested review from albanD, antoniojkim, bdhirsh, miladm and SherlockNoMad November 27, 2024 13:49

ezyang mentioned this pull request Nov 27, 2024

Unify cache disable and cache bypass paths #141685

Closed

ezyang requested review from jansel, jamesjwu and masnesral November 27, 2024 14:04

ezyang added the topic: not user facing topic category label Nov 27, 2024

jamesjwu approved these changes Nov 27, 2024

View reviewed changes

ezyang added a commit that referenced this pull request Nov 27, 2024

Hoist set_feature_use out of conditional, rename some variables

7e28e4a

Signed-off-by: Edward Z. Yang <ezyang@meta.com> ghstack-source-id: 9afb56f Pull Request resolved: #141683

jansel approved these changes Nov 27, 2024

View reviewed changes

Update

cf47986

[ghstack-poisoned]

pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 27, 2024

This was referenced Nov 28, 2024

Unify post_compile1 and CompiledFxGraph constructor #141689

Closed

Inline compile_to_fn at its only call site #141691

Closed

Introduce CompiledAOTI #141695

Closed

pytorchmergebot added the merging label Nov 28, 2024

pytorchmergebot removed the merging label Nov 28, 2024

Update

6f35720

[ghstack-poisoned]

ezyang mentioned this pull request Nov 28, 2024

[POC] AOTInductor as Inductor backend #141700

Closed

pytorchmergebot added the merging label Nov 28, 2024

pytorchmergebot added the Merged label Nov 28, 2024

pytorchmergebot closed this in 6d204cb Nov 28, 2024

pytorchmergebot removed the merging label Nov 28, 2024

github-actions bot deleted the gh/ezyang/3024/head branch December 30, 2024 02:08

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Hoist set_feature_use out of conditional, rename some variables #141683

Hoist set_feature_use out of conditional, rename some variables #141683

Uh oh!

ezyang commented Nov 27, 2024 •

edited

Loading

Uh oh!

pytorch-bot bot commented Nov 27, 2024 •

edited

Loading

Uh oh!

ezyang commented Nov 27, 2024

Uh oh!

ezyang commented Nov 28, 2024

Uh oh!

pytorchmergebot commented Nov 28, 2024

Uh oh!

pytorchmergebot commented Nov 28, 2024

Uh oh!

ezyang commented Nov 28, 2024

Uh oh!

pytorchmergebot commented Nov 28, 2024

Uh oh!

Uh oh!

Hoist set_feature_use out of conditional, rename some variables #141683

Hoist set_feature_use out of conditional, rename some variables #141683

Uh oh!

Conversation

ezyang commented Nov 27, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pytorch-bot bot commented Nov 27, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🔗 Helpful Links

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

✅ You can merge normally! (2 Unrelated Failures)

Uh oh!

ezyang commented Nov 27, 2024

Uh oh!

ezyang commented Nov 28, 2024

Uh oh!

pytorchmergebot commented Nov 28, 2024

Merge started

Uh oh!

pytorchmergebot commented Nov 28, 2024

Merge failed

Uh oh!

ezyang commented Nov 28, 2024

Uh oh!

pytorchmergebot commented Nov 28, 2024

Merge started

Uh oh!

Uh oh!

ezyang commented Nov 27, 2024 •

edited

Loading

pytorch-bot bot commented Nov 27, 2024 •

edited

Loading