Skip to content

Revert "symbolic shapes guarding_hint_or_throw and optimization_hint (#175497)"#177113

Closed
laithsakka wants to merge 1 commit intogh/laithsakka/423/basefrom
gh/laithsakka/423/head
Closed

Revert "symbolic shapes guarding_hint_or_throw and optimization_hint (#175497)"#177113
laithsakka wants to merge 1 commit intogh/laithsakka/423/basefrom
gh/laithsakka/423/head

Conversation

@laithsakka
Copy link
Contributor

@laithsakka laithsakka commented Mar 11, 2026

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 11, 2026

🔗 Helpful Links

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

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

✅ You can merge normally! (7 Unrelated Failures)

As of commit aba7e46 with merge base 4bc9d7f (image):

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

BROKEN TRUNK - The following jobs failed but were 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.

@pytorch-bot pytorch-bot bot added ci-no-td Do not run TD on this PR ciflow/inductor ciflow/torchtitan Run TorchTitan integration tests module: dynamo labels 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
@pytorch-bot pytorch-bot bot added module: inductor ci-no-td Do not run TD on this PR release notes: fx release notes category ciflow/inductor ciflow/torchtitan Run TorchTitan integration tests module: dynamo labels Mar 11, 2026
@laithsakka laithsakka added the suppress-bc-linter Suppresses the failures of API backward-compatibility linter (Lint/bc_linter) label Mar 11, 2026
@laithsakka laithsakka requested a review from huydhn March 11, 2026 04:11
@huydhn
Copy link
Contributor

huydhn commented Mar 11, 2026

@pytorchbot merge

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 11, 2026

This PR needs to be approved by an authorized maintainer before merge.

@huydhn
Copy link
Contributor

huydhn commented Mar 11, 2026

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Mar 11, 2026
@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

@pytorchmergebot
Copy link
Collaborator

The merge job was canceled or timed out. This most often happen if two merge requests were issued for the same PR, or if merge job was waiting for more than 6 hours for tests to finish. In later case, please do not hesitate to reissue the merge command
For more information see pytorch-bot wiki.

@huydhn
Copy link
Contributor

huydhn commented Mar 11, 2026

@pytorchbot merge

@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

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/torchtitan Run TorchTitan integration tests ciflow/trunk Trigger trunk jobs on your pull request Merged module: dynamo module: inductor release notes: fx release notes category 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.

3 participants