Skip to content

Conversation

malfet
Copy link
Contributor

@malfet malfet commented Sep 23, 2025

Stack from ghstack (oldest at bottom):

I.e. use TimeoutError instead of socket.timeout

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Sep 23, 2025

🔗 Helpful Links

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

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

⏳ No Failures, 24 Pending

As of commit f529b4a with merge base 649ceda (image):
💚 Looks good so far! There are no failures yet. 💚

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

This was referenced Sep 23, 2025
[ghstack-poisoned]
@malfet malfet requested a review from a team as a code owner September 23, 2025 17:54
[ghstack-poisoned]
@pytorchmergebot
Copy link
Collaborator

Starting merge as part of PR stack under #163649

pytorchmergebot pushed a commit that referenced this pull request Sep 23, 2025
As it has been removed, see https://docs.astral.sh/ruff/rules/non-pep604-isinstance/
Pull Request resolved: #163649
Approved by: https://github.com/Skylion007
ghstack dependencies: #163648
pytorchmergebot pushed a commit that referenced this pull request Sep 23, 2025
Pull Request resolved: #163653
Approved by: https://github.com/jansel
ghstack dependencies: #163648, #163649
dsashidh pushed a commit to dsashidh/pytorch that referenced this pull request Sep 26, 2025
I.e. use `TimeoutError` instead of `socket.timeout`
Pull Request resolved: pytorch#163648
Approved by: https://github.com/cyyever, https://github.com/Skylion007
dsashidh pushed a commit to dsashidh/pytorch that referenced this pull request Sep 26, 2025
dsashidh pushed a commit to dsashidh/pytorch that referenced this pull request Sep 26, 2025
jainapurva pushed a commit that referenced this pull request Sep 29, 2025
Pull Request resolved: #163653
Approved by: https://github.com/jansel
ghstack dependencies: #163648, #163649
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.

4 participants