Skip to content

Conversation

@izdeby
Copy link
Contributor

@izdeby izdeby commented Aug 5, 2019

Summary:
Pull Request resolved: #22261
ghimport-source-id: 1611d62

Test Plan: Imported from OSS

Differential Revision: D16005990

Pulled By: izdeby

fbshipit-source-id: 2413824aa75a0755719e4df11acd21e6607e5a85

…sor (#22261)

Summary:
Pull Request resolved: #22261
ghimport-source-id: 1611d62

Test Plan: Imported from OSS

Differential Revision: D16005990

Pulled By: izdeby

fbshipit-source-id: 2413824aa75a0755719e4df11acd21e6607e5a85
@pytorchbot pytorchbot added module: cuda Related to torch.cuda, and CUDA support in general module: docs Related to our documentation, both in docs/ and docblocks module: internals Related to internal abstractions in c10 and ATen module: operators module: typing Related to mypy type annotations labels Aug 5, 2019
@izdeby izdeby mentioned this pull request Aug 5, 2019
for (const auto & index : indices) {
if (index.scalar_type() == kByte || index.scalar_type() == kBool) {
if (index.scalar_type() == kByte) {
AT_WARN("indexing with dtype torch.uint8 is now deprecated," \
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn’t it be appropriate to use TORCH_WARN instead of the deprecated AT_WARN?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Its a cherry-pick from the master branch. If this has to be fixed, it should be done there first.

@izdeby izdeby requested a review from soumith August 5, 2019 21:11
@soumith soumith closed this Aug 7, 2019
@soumith soumith reopened this Aug 7, 2019
@soumith soumith merged commit 060f677 into v1.2.0 Aug 7, 2019
@facebook-github-bot facebook-github-bot deleted the mergeDocs branch July 13, 2020 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: cuda Related to torch.cuda, and CUDA support in general module: docs Related to our documentation, both in docs/ and docblocks module: internals Related to internal abstractions in c10 and ATen module: typing Related to mypy type annotations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants