Skip to content

[UNSTACK] Replace simple if_true / if_false cases in Declarations.cwrap.#26285

Closed
gchanan wants to merge 1 commit intogh/gchanan/84/basefrom
gh/gchanan/84/head
Closed

[UNSTACK] Replace simple if_true / if_false cases in Declarations.cwrap.#26285
gchanan wants to merge 1 commit intogh/gchanan/84/basefrom
gh/gchanan/84/head

Conversation

@gchanan
Copy link
Copy Markdown
Contributor

@gchanan gchanan commented Sep 16, 2019

Stack from ghstack:

I renamed:
THTensor_(std / var) -> THTensor(std_single / var_single)
THTensor(stdall / varall) -> THTensor(std_all, var_all)

because I reversed the meaning of the bias/unbiased parameters (to match ATen) and type checking wouldn't catch failures.

Differential Revision: D17397227

I renamed:
THTensor_(std / var) -> THTensor(std_single / var_single)
THTensor(stdall / varall) -> THTensor(std_all, var_all)

because I reversed the meaning of the bias/unbiased parameters (to match ATen) and type checking wouldn't catch failures.
@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 16, 2019
gchanan added a commit that referenced this pull request Sep 16, 2019
I renamed:
THTensor_(std / var) -> THTensor(std_single / var_single)
THTensor(stdall / varall) -> THTensor(std_all, var_all)

because I reversed the meaning of the bias/unbiased parameters (to match ATen) and type checking wouldn't catch failures.

ghstack-source-id: ad17f35
Pull Request resolved: #26285
@gchanan gchanan requested a review from zou3519 September 16, 2019 15:07
@gchanan
Copy link
Copy Markdown
Contributor Author

gchanan commented Sep 16, 2019

Same as #25614.

zdevito pushed a commit to zdevito/ATen that referenced this pull request Sep 17, 2019
Summary:
Pull Request resolved: pytorch/pytorch#26285

I renamed:
THTensor_(std / var) -> THTensor(std_single / var_single)
THTensor(stdall / varall) -> THTensor(std_all, var_all)

because I reversed the meaning of the bias/unbiased parameters (to match ATen) and type checking wouldn't catch failures.

Test Plan: Imported from OSS

Differential Revision: D17397227

Pulled By: gchanan

fbshipit-source-id: 244fe878d4e1045620137c00fbaea6e6f919fc8d
@facebook-github-bot
Copy link
Copy Markdown
Contributor

@gchanan merged this pull request in a3915bd.

@facebook-github-bot facebook-github-bot deleted the gh/gchanan/84/head branch October 28, 2019 22:13
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
…ch#26285)

Summary:
Pull Request resolved: pytorch#26285

I renamed:
THTensor_(std / var) -> THTensor(std_single / var_single)
THTensor(stdall / varall) -> THTensor(std_all, var_all)

because I reversed the meaning of the bias/unbiased parameters (to match ATen) and type checking wouldn't catch failures.

Test Plan: Imported from OSS

Differential Revision: D17397227

Pulled By: gchanan

fbshipit-source-id: 244fe878d4e1045620137c00fbaea6e6f919fc8d
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