Skip to content

Conversation

swolchok
Copy link
Contributor

@swolchok swolchok commented Aug 25, 2025

Stack from ghstack (oldest at bottom):

If I am reading THPUtils_checkIndex correctly, it already checks torch::is_symint before hitting index or int.

If I am reading [THPUtils_checkIndex](https://github.com/pytorch/pytorch/blob/cf94cadbeee31a4d1d46a57f11bce7c9fd1cebc0/torch/csrc/utils.cpp#L28) correctly, it already checks torch::is_symint before hitting __index__ or __int__.

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Aug 25, 2025

🔗 Helpful Links

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

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

❌ 10 New Failures

As of commit 275b500 with merge base ffa1ce7 (image):

NEW FAILURES - The following jobs have failed:

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

swolchok added a commit that referenced this pull request Aug 25, 2025
If I am reading [THPUtils_checkIndex](https://github.com/pytorch/pytorch/blob/cf94cadbeee31a4d1d46a57f11bce7c9fd1cebc0/torch/csrc/utils.cpp#L28) correctly, it already checks torch::is_symint before hitting __index__ or __int__.

ghstack-source-id: f31e67a
Pull Request resolved: #161438
@swolchok swolchok added the topic: not user facing topic category label Aug 25, 2025
@swolchok
Copy link
Contributor Author

abandoning; I think we want to add a check for plain old int, not delete checks

@swolchok swolchok closed this Aug 27, 2025
@github-actions github-actions bot deleted the gh/swolchok/806/head branch September 27, 2025 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant