-
Notifications
You must be signed in to change notification settings - Fork 26.3k
Add operator<< overloads for TensorOptions #9606
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
facebook-github-bot
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goldsborough has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
|
@pytorchbot retest this please |
2 similar comments
|
@pytorchbot retest this please |
|
@pytorchbot retest this please |
da52faa to
2b320e4
Compare
aten/src/ATen/Layout.h
Outdated
| } // namespace at | ||
|
|
||
| inline std::ostream& operator<<(std::ostream& stream, at::Layout layout) { | ||
| return stream << ((layout == at::kStrided) ? "Strided" : "Sparse"); |
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
ezyang
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved with nits fixed.
facebook-github-bot
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@goldsborough is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Summary: Added `operator<<` overloads for `at::TensorOptions` on request of ebetica Example output: ``` TensorOptions(dtype=Double, device=cpu, layout=Strided, requires_grad=false) ``` ezyang apaszke Pull Request resolved: pytorch/pytorch#9606 Differential Revision: D8925191 Pulled By: goldsborough fbshipit-source-id: 0503bc2851268276e9561d918290bc723e437c9c
Summary: Added `operator<<` overloads for `at::TensorOptions` on request of ebetica Example output: ``` TensorOptions(dtype=Double, device=cpu, layout=Strided, requires_grad=false) ``` ezyang apaszke Pull Request resolved: pytorch#9606 Differential Revision: D8925191 Pulled By: goldsborough fbshipit-source-id: 0503bc2851268276e9561d918290bc723e437c9c
Summary: Added `operator<<` overloads for `at::TensorOptions` on request of ebetica Example output: ``` TensorOptions(dtype=Double, device=cpu, layout=Strided, requires_grad=false) ``` ezyang apaszke Pull Request resolved: pytorch#9606 Differential Revision: D8925191 Pulled By: goldsborough fbshipit-source-id: 0503bc2851268276e9561d918290bc723e437c9c
Added
operator<<overloads forat::TensorOptionson request of @ebeticaExample output:
@ezyang @apaszke