Skip to content

Removes torchtest, expands generic device testing#26374

Closed
mruberry wants to merge 3 commits intomasterfrom
test_nn_device_generic
Closed

Removes torchtest, expands generic device testing#26374
mruberry wants to merge 3 commits intomasterfrom
test_nn_device_generic

Conversation

@mruberry
Copy link
Copy Markdown
Collaborator

@mruberry mruberry commented Sep 17, 2019

  • Removes torchtest
  • Moves test_torch tests skipped on ROCm to generic device test class
  • Creates test_nn generic device test class

Next: adding @dtypes to generic device testing framework.

@pytorchbot pytorchbot added module: cuda Related to torch.cuda, and CUDA support in general module: nn Related to torch.nn module: operators module: tests Issues related to tests (not the torch.testing module) labels Sep 17, 2019
@mruberry mruberry requested review from ailzhang and izdeby September 17, 2019 22:18
Copy link
Copy Markdown
Contributor

@ailzhang ailzhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mruberry
Copy link
Copy Markdown
Collaborator Author

ROCm build is hanging just as it was in a previous PR. After successfully completing TestCUDA it simply stops running. This behavior is strange since in both this and the previous PR tests were only removed, not added to, TestCUDA.

I'll revert some of the ROCm-related changes.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mruberry has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@mruberry mruberry deleted the test_nn_device_generic branch September 18, 2019 17:35
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@mruberry merged this pull request in 388cfdf.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
- Removes torchtest
- <s>Moves test_torch tests skipped on ROCm to generic device test class</s>
- Creates test_nn generic device test class

Next: adding dtypes to generic device testing framework.
Pull Request resolved: pytorch#26374

Test Plan: Change is to tests themselves.

Differential Revision: D17442218

Pulled By: mruberry

fbshipit-source-id: d7e4451d09fc9049478b35a7efb8bb580071e8c8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: cuda Related to torch.cuda, and CUDA support in general module: nn Related to torch.nn module: tests Issues related to tests (not the torch.testing module)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants