Skip to content

unxfail passing test#2220

Merged
t-vi merged 1 commit intomainfrom
tom/default_device_with_ctx
Jun 11, 2025
Merged

unxfail passing test#2220
t-vi merged 1 commit intomainfrom
tom/default_device_with_ctx

Conversation

@t-vi
Copy link
Copy Markdown
Collaborator

@t-vi t-vi commented Jun 11, 2025

Before submitting
  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • n/a Did you make sure to update the docs?
  • Did you write any new necessary tests?

Recently, @kshitij12345 fixed get_default_device appreciating with device: .:rocket:
So we unxfail the test for future versions of PyTorch.

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@t-vi t-vi requested review from lantiga and mruberry as code owners June 11, 2025 18:32
@t-vi t-vi merged commit c585428 into main Jun 11, 2025
49 checks passed
@t-vi t-vi deleted the tom/default_device_with_ctx branch June 11, 2025 19:16
@nWEIdia
Copy link
Copy Markdown
Collaborator

nWEIdia commented Jul 11, 2025

@t-vi Could you please provide a link to "fixed get_default_device appreciating with device" ?

@kshitij12345
Copy link
Copy Markdown
Collaborator

I think it was this PR pytorch/pytorch#148621

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants