Skip to content

deprecate check_is_size and guard_size_oblivious (#167198)#169400

Closed
laithsakka wants to merge 1 commit intopytorch:mainfrom
laithsakka:export-D88186600
Closed

deprecate check_is_size and guard_size_oblivious (#167198)#169400
laithsakka wants to merge 1 commit intopytorch:mainfrom
laithsakka:export-D88186600

Conversation

@laithsakka
Copy link
Contributor

@laithsakka laithsakka commented Dec 2, 2025

@meta-codesync
Copy link

meta-codesync bot commented Dec 2, 2025

@laithsakka has exported this pull request. If you are a Meta employee, you can view the originating Diff in D88186600.

@pytorch-bot
Copy link

pytorch-bot bot commented Dec 2, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit e1b9381 with merge base 7ea27ac (image):

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

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

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Dec 5, 2025
@Skylion007
Copy link
Collaborator

Do you want the warning category to be a DeprecatedWarning Category or a FutureWarning. Ie do you want to yell at people in IDEs opted into it or end users because it's about to be removed. If so change the kwarg for the decorator to reflect it

pytorch-bot bot pushed a commit that referenced this pull request Jan 6, 2026
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 6, 2026
@laithsakka laithsakka force-pushed the export-D88186600 branch 2 times, most recently from 3bdb67d to c9135ee Compare January 7, 2026 00:55
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
@laithsakka laithsakka force-pushed the export-D88186600 branch 2 times, most recently from ebba9cf to 4b1d512 Compare January 7, 2026 16:12
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
…torch#169400)

Summary:

title

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/f2e6f94081c02704adf027fe0c81bf25726828f5

Reviewed By: svalentin, Mingming-Ding

Differential Revision: D88186600
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
…torch#169400)

Summary:

title

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/f2e6f94081c02704adf027fe0c81bf25726828f5

Reviewed By: svalentin, Mingming-Ding

Differential Revision: D88186600
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
…torch#169400)

Summary:

title

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/f2e6f94081c02704adf027fe0c81bf25726828f5

Reviewed By: svalentin, Mingming-Ding

Differential Revision: D88186600
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
…torch#169400)

Summary:

title

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/f2e6f94081c02704adf027fe0c81bf25726828f5

Reviewed By: svalentin, Mingming-Ding

Differential Revision: D88186600
laithsakka added a commit to laithsakka/pytorch that referenced this pull request Jan 7, 2026
…torch#169400)

Summary:

title

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/f2e6f94081c02704adf027fe0c81bf25726828f5

Reviewed By: svalentin, Mingming-Ding

Differential Revision: D88186600
…torch#169400)

Summary:

title

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/f2e6f94081c02704adf027fe0c81bf25726828f5

Reviewed By: svalentin, Mingming-Ding

Differential Revision: D88186600
@facebook-github-bot
Copy link
Contributor

@pytorchbot merge

(Initiating merge automatically since Phabricator Diff has merged)

@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

hinriksnaer pushed a commit to hinriksnaer/pytorch that referenced this pull request Jan 12, 2026
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