Skip to content

Named tensor support for: index_fill_, index_fill, squeeze, median(Tensor)#26914

Closed
zou3519 wants to merge 5 commits intogh/zou3519/192/basefrom
gh/zou3519/192/head
Closed

Named tensor support for: index_fill_, index_fill, squeeze, median(Tensor)#26914
zou3519 wants to merge 5 commits intogh/zou3519/192/basefrom
gh/zou3519/192/head

Conversation

@zou3519
Copy link
Copy Markdown
Contributor

@zou3519 zou3519 commented Sep 26, 2019

Stack from ghstack:

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:

  • [namedtensor ci]

Differential Revision: D17609136

…nsor)

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:
- [namedtensor ci]
@pytorchbot pytorchbot added module: cpu CPU specific problem (e.g., perf, algorithm) module: cuda Related to torch.cuda, and CUDA support in general module: internals Related to internal abstractions in c10 and ATen module: operators labels Sep 26, 2019
@zou3519 zou3519 requested a review from nairbv September 26, 2019 19:54
Comment thread aten/src/ATen/native/TensorShape.cpp Outdated
}

#ifdef BUILD_NAMEDTENSOR
optional<std::vector<Dimname>> inferSqueezeNames(const Tensor& tensor) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

should this be in NamedTensorUtils.cpp?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sure

@zou3519 zou3519 added this to the 1.3 milestone Sep 26, 2019
…, median(Tensor)"

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:
- [namedtensor ci]

Differential Revision: [D17609136](https://our.internmc.facebook.com/intern/diff/D17609136)
zou3519 added a commit that referenced this pull request Sep 26, 2019
…nsor)

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:
- [namedtensor ci]

ghstack-source-id: 18de0a5
Pull Request resolved: #26914
…, median(Tensor)"

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:
- [namedtensor ci]

Differential Revision: [D17609136](https://our.internmc.facebook.com/intern/diff/D17609136)
…, median(Tensor)"

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:
- [namedtensor ci]

Differential Revision: [D17609136](https://our.internmc.facebook.com/intern/diff/D17609136)
zou3519 added a commit that referenced this pull request Sep 27, 2019
…nsor)

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:
- [namedtensor ci]

ghstack-source-id: 5edf2e7
Pull Request resolved: #26914
…, median(Tensor)"

Also added dimname overloads for index_fill_ and squeeze.

Test Plan:
- [namedtensor ci]

Differential Revision: [D17609136](https://our.internmc.facebook.com/intern/diff/D17609136)
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@zou3519 merged this pull request in 3ad1bbe.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Sep 27, 2019
…nsor) (#26914)

Summary:
Pull Request resolved: pytorch/pytorch#26914

Also added dimname overloads for index_fill_ and squeeze.

Test Plan: - [namedtensor ci]

Differential Revision: D17609136

Pulled By: zou3519

fbshipit-source-id: 29c7ad52ffe24e0b3ad679111fee7a78eca7acdf
@facebook-github-bot facebook-github-bot deleted the gh/zou3519/192/head branch October 28, 2019 22:24
pdlive215 pushed a commit to pdlive215/pytorch that referenced this pull request Nov 27, 2019
…nsor) (pytorch#26914)

Summary:
Pull Request resolved: pytorch#26914

Also added dimname overloads for index_fill_ and squeeze.

Test Plan: - [namedtensor ci]

Differential Revision: D17609136

Pulled By: zou3519

fbshipit-source-id: 29c7ad52ffe24e0b3ad679111fee7a78eca7acdf
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
…nsor) (pytorch#26914)

Summary:
Pull Request resolved: pytorch#26914

Also added dimname overloads for index_fill_ and squeeze.

Test Plan: - [namedtensor ci]

Differential Revision: D17609136

Pulled By: zou3519

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

Labels

Merged module: cpu CPU specific problem (e.g., perf, algorithm) module: cuda Related to torch.cuda, and CUDA support in general module: internals Related to internal abstractions in c10 and ATen

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants