Skip to content

Conversation

datumbox
Copy link
Contributor

Summary:
This diff ports quantize_val and dequantize_val from at::native to torchvision because native kernels are incompatible with android xplat builds (see D30234056).

This should only be temporary until we find a way to move those functions out of at::native, or until the at::native / android incompatibility disappears.

Reviewed By: fmassa

Differential Revision: D30393619

fbshipit-source-id: 18b7b1b349ad9a24088a120e23da7535f7fa7ddc

…tive and android xplat incompatibility

Summary:
This diff ports `quantize_val` and `dequantize_val` from at::native to torchvision because native kernels are incompatible with android xplat builds (see D30234056).

This should only be temporary until we find a way to move those functions out of at::native, or until the at::native / android incompatibility disappears.

Reviewed By: fmassa

Differential Revision: D30393619

fbshipit-source-id: 18b7b1b349ad9a24088a120e23da7535f7fa7ddc
@datumbox
Copy link
Contributor Author

This PR is a sync from FBcode. I suggest merging it even if it breaks master due to clang formatting issues and fix ASAP on a follow up PR. This will simplify the process of syncing the change back to FBcode later on.

Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

Thanks!

@datumbox datumbox merged commit 32f95c7 into pytorch:main Aug 26, 2021
@datumbox datumbox deleted the xplat_incompatibility branch August 26, 2021 08:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants