Skip to content

Dispatch aten::maximum and aten::minimum to XLATensor::max and XLATen…#2438

Merged
JackCaoG merged 2 commits intomasterfrom
maximum_minimum_other_node
Aug 27, 2020
Merged

Dispatch aten::maximum and aten::minimum to XLATensor::max and XLATen…#2438
JackCaoG merged 2 commits intomasterfrom
maximum_minimum_other_node

Conversation

@JackCaoG
Copy link
Copy Markdown
Collaborator

@JackCaoG JackCaoG commented Aug 18, 2020

…sor::min

Fixes #2424. Make maximum and minimum perform pytorch type promotion too.

@JackCaoG JackCaoG requested review from ailzhang and davidel August 18, 2020 01:47
@JackCaoG
Copy link
Copy Markdown
Collaborator Author

pytorch change pytorch/pytorch#42579 doesn't have the AutogradXLA renaming change, hence the build will fail. I will ask if they can rebase. I patched the pytorch change and run python tests locally. All of the affected test passed.

@JackCaoG JackCaoG force-pushed the maximum_minimum_other_node branch from 8cfef42 to 09ba6c4 Compare August 20, 2020 01:49
@JackCaoG
Copy link
Copy Markdown
Collaborator Author

Dsiable test_maximum_minimum_float_nan_and_inf on Xla:GPU because it has a different behavior on maximum(nan, inf) than XLA:CPU and XLA:GPU (and pytorch native).

@mruberry
Copy link
Copy Markdown
Contributor

Hey PyTorch/XLA team - is this OK to go in? We'd like to unblock pytorch/pytorch#42579.

@JackCaoG
Copy link
Copy Markdown
Collaborator Author

@mruberry yup, I will remove the pin and merge this pr after pytorch one merged.

@JackCaoG JackCaoG force-pushed the maximum_minimum_other_node branch from 09ba6c4 to 6f675c8 Compare August 27, 2020 00:11
@JackCaoG JackCaoG merged commit bde3f5c into master Aug 27, 2020
cjolivier01 pushed a commit to cjolivier01/pytorch_xla that referenced this pull request Mar 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[PT_BREAK] implement NumPy-like functionality maximum, minimum

3 participants