Skip to content

symbolic shapes guarding_hint_or_throw and optimization_hint#175497

Closed
laithsakka wants to merge 24 commits intogh/laithsakka/406/basefrom
gh/laithsakka/406/head
Closed

symbolic shapes guarding_hint_or_throw and optimization_hint#175497
laithsakka wants to merge 24 commits intogh/laithsakka/406/basefrom
gh/laithsakka/406/head

Conversation

@laithsakka
Copy link
Contributor

@laithsakka laithsakka commented Feb 22, 2026

@pytorch-bot
Copy link

pytorch-bot bot commented Feb 22, 2026

🔗 Helpful Links

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

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

✅ You can merge normally! (3 Unrelated Failures)

As of commit fad6bf3 with merge base 07efc60 (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

BROKEN TRUNK - The following job failed but was present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

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

cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx ipiszy kadeng muchulee8 amjames chauhang aakhundov coconutruben jataylo Lucaskabela

[ghstack-poisoned]
laithsakka added a commit that referenced this pull request Feb 22, 2026
ghstack-source-id: e299aef
Pull Request resolved: #175497
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx ipiszy kadeng muchulee8 amjames chauhang aakhundov coconutruben jataylo Lucaskabela

[ghstack-poisoned]
laithsakka added a commit that referenced this pull request Feb 22, 2026
ghstack-source-id: 0eeb78e
Pull Request resolved: #175497
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx ipiszy kadeng muchulee8 amjames chauhang aakhundov coconutruben jataylo Lucaskabela

[ghstack-poisoned]
laithsakka added a commit that referenced this pull request Feb 24, 2026
ghstack-source-id: 8334b3a
Pull Request resolved: #175497
cc voznesenskym penguinwu EikanWang jgong5 Guobing-Chen XiaobingSuper zhuhaozhe blzheng wenzhe-nrv jiayisunx ipiszy kadeng muchulee8 amjames chauhang aakhundov coconutruben jataylo Lucaskabela

[ghstack-poisoned]
laithsakka added a commit that referenced this pull request Feb 24, 2026
ghstack-source-id: dc8a5fa
Pull Request resolved: #175497
strict_zip = partial(zip, strict=True)


def _get_symint_hints(exprs: Any) -> Any:
Copy link
Contributor Author

Choose a reason for hiding this comment

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

unused

@pytorchmergebot
Copy link
Collaborator

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

@zou3519
Copy link
Contributor

zou3519 commented Mar 7, 2026

@pytorchbot revert -c "nosignal" -m "breaking compile time benchmarks"

@zou3519
Copy link
Contributor

zou3519 commented Mar 7, 2026

image

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

pytorchmergebot added a commit that referenced this pull request Mar 7, 2026
…175497)"

This reverts commit 5eb1752.

Reverted #175497 on behalf of https://github.com/zou3519 due to breaking compile time benchmarks ([comment](#175497 (comment)))
@pytorchmergebot
Copy link
Collaborator

@laithsakka your PR has been successfully reverted.

@laithsakka
Copy link
Contributor Author

@pytorchbot revert -c "nosignal" -m "breaking compile time benchmarks"

no it was not me :( it was this PR
615c79f
@zou3519 I messaged him he have a forward fix.
#176772

@laithsakka
Copy link
Contributor Author

laithsakka commented Mar 7, 2026

I will land this first #176755 to unblock landing my PR
once the fix for that regression (#176772) lands it shall reset the expected numbers back

@laithsakka
Copy link
Contributor Author

@pytorchbot merge -f "already landed a fix for the pr_time benchmarks, the reason for revert see comment above"

@pytorchmergebot
Copy link
Collaborator

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

@laithsakka
Copy link
Contributor Author

@pytorchbot revert -c "internal errors :("

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 10, 2026

❌ 🤖 pytorchbot command failed:

@pytorchbot revert: error: argument -c/--classification: invalid choice: 'internal errors :(' (choose from 'nosignal', 'ignoredsignal', 'landrace', 'weird', 'ghfirst', 'autorevert')

usage: @pytorchbot revert -m MESSAGE -c
                          {nosignal,ignoredsignal,landrace,weird,ghfirst,autorevert}

Try @pytorchbot --help for more info.

@laithsakka
Copy link
Contributor Author

@pytorchbot revert -c "nosignal" -m "internal errors"

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

@pytorchmergebot
Copy link
Collaborator

Reverting PR 175497 failed

Reason: Command git -C /home/runner/work/pytorch/pytorch revert --no-edit 9cc969af7598c6e9a11d46646c64e4e083445665 returned non-zero exit code 1

Auto-merging test/test_dynamic_shapes.py
Auto-merging torch/_dynamo/trace_rules.py
Auto-merging torch/_dynamo/variables/torch.py
Auto-merging torch/_functorch/aot_autograd.py
Auto-merging torch/_inductor/codecache.py
Auto-merging torch/_inductor/lowering.py
Auto-merging torch/_inductor/sizevars.py
CONFLICT (content): Merge conflict in torch/_inductor/sizevars.py
Auto-merging torch/_refs/__init__.py
Auto-merging torch/_subclasses/fake_impls.py
CONFLICT (modify/delete): torch/fx/experimental/_size_hinting.py deleted in parent of 9cc969af759 (symbolic shapes guarding_hint_or_throw and optimization_hint (#175497)) and modified in HEAD.  Version HEAD of torch/fx/experimental/_size_hinting.py left in tree.
Auto-merging torch/fx/experimental/symbolic_shapes.py
CONFLICT (content): Merge conflict in torch/fx/experimental/symbolic_shapes.py
error: could not revert 9cc969af759... symbolic shapes guarding_hint_or_throw and optimization_hint (#175497)
hint: After resolving the conflicts, mark them with
hint: "git add/rm <pathspec>", then run
hint: "git revert --continue".
hint: You can instead skip this commit with "git revert --skip".
hint: To abort and get back to the state before "git revert",
hint: run "git revert --abort".
hint: Disable this message with "git config set advice.mergeConflict false"
Details for Dev Infra team Raised by workflow job

laithsakka added a commit that referenced this pull request Mar 11, 2026
laithsakka added a commit that referenced this pull request Mar 11, 2026
…175497)"

This reverts commit 9cc969a.

ghstack-source-id: 0392a62
Pull Request resolved: #177112
laithsakka added a commit that referenced this pull request Mar 11, 2026
laithsakka added a commit that referenced this pull request Mar 11, 2026
…175497)"

This reverts commit 9cc969a.

ghstack-source-id: 12278ce
Pull Request resolved: #177113
pytorchmergebot pushed a commit that referenced this pull request Mar 11, 2026
…175497)" (#177113)

This reverts commit 9cc969a.
Due to internal issues will re-land from fbcode.

Pull Request resolved: #177113
Approved by: https://github.com/huydhn
pytorchmergebot pushed a commit that referenced this pull request Mar 11, 2026
…#177184)

This reverts commit b244229.

This is needed to unblock diff train because #176918 has a merge conflict with #175497 and the latter has been reverted by #177113. The PRs and their reverts were landed in interlocking order in fbcode, so this is the only way to untangle them.

* `torch/fx/experimental/_size_hinting.py`
* `torch/fx/experimental/symbolic_shapes.py`

We probably to coordinate their landing later
Pull Request resolved: #177184
Approved by: https://github.com/seemethere, https://github.com/zou3519
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-no-td Do not run TD on this PR ciflow/inductor ciflow/trunk Trigger trunk jobs on your pull request keep-going Don't stop on first failure, keep running tests until the end Merged module: dynamo module: inductor release notes: jit release notes category Reverted suppress-bc-linter Suppresses the failures of API backward-compatibility linter (Lint/bc_linter)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants