Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

Conversation

@corey-nm
Copy link
Contributor

@corey-nm corey-nm commented Nov 3, 2022

See comments in code for more details.

Related to #604

Test plan: added unit tests and ran on torch 1.9 and 1.12.0. The GHW will also run this test on both versions.

@corey-nm corey-nm changed the title Ensuring has_custimzed_qrange==False for torch <=1.9.1 and >=1.12.0 Ensuring proper has_custimzed_qrange for torch <=1.9.1 and >=1.12.0 Nov 3, 2022
bfineran
bfineran previously approved these changes Nov 4, 2022
Copy link
Contributor

@bfineran bfineran left a comment

Choose a reason for hiding this comment

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

tests look good

dbogunowicz
dbogunowicz previously approved these changes Nov 7, 2022
Copy link
Contributor

@dbogunowicz dbogunowicz left a comment

Choose a reason for hiding this comment

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

one nit in comments

@corey-nm corey-nm dismissed stale reviews from dbogunowicz and bfineran via b2e0f17 November 8, 2022 16:14
Copy link
Contributor

@KSGulin KSGulin left a comment

Choose a reason for hiding this comment

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

Really nice solution 🙂

@corey-nm corey-nm merged commit 2c36d24 into main Nov 8, 2022
@corey-nm corey-nm deleted the zero-point-128-torch-1.12 branch November 8, 2022 16:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants