Skip to content

Better error message for quantized dispatch #28635

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
z-a-f wants to merge 9 commits into from
Closed

Better error message for quantized dispatch #28635

z-a-f wants to merge 9 commits into from

Conversation

@z-a-f
Copy link

@z-a-f z-a-f commented Oct 25, 2019 •
edited
Loading

Stack from ghstack:

Fixes #28518

Differential Revision: D18132566

z-a-f pushed a commit that referenced this pull request Oct 25, 2019
Better error message for quantized dispatch
79b383e 
Fixes #28518

ghstack-source-id: a1c60d0
Pull Request resolved: #28635
@z-a-f z-a-f added the oncall: quantization Quantization support in PyTorch label Oct 25, 2019
@z-a-f z-a-f requested review from jamesr66a and jerryzh168 October 25, 2019 03:27
z-a-f pushed a commit that referenced this pull request Oct 25, 2019
Better error message for quantized dispatch
f711ba8 
Fixes #28518

ghstack-source-id: e82c911
Pull Request resolved: #28635
jamesr66a
jamesr66a reviewed Oct 25, 2019
View reviewed changes
Copy link
Collaborator

@jamesr66a jamesr66a left a comment

Choose a reason for hiding this comment

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

Thanks! Left some small comments

cc @smessmer

jerryzh168
jerryzh168 reviewed Oct 25, 2019
View reviewed changes
z-a-f pushed a commit that referenced this pull request Oct 25, 2019
Better error message for quantized dispatch
6b1a18d 
Fixes #28518

ghstack-source-id: 67896d6
Pull Request resolved: #28635
z-a-f pushed a commit that referenced this pull request Oct 28, 2019
Better error message for quantized dispatch
0c0a547 
Fixes #28518

ghstack-source-id: 1c064ff
Pull Request resolved: #28635
@z-a-f z-a-f requested review from jamesr66a and jerryzh168 October 28, 2019 20:02
jianyuh
jianyuh approved these changes Oct 29, 2019
View reviewed changes
Copy link
Member

@jianyuh jianyuh left a comment

Choose a reason for hiding this comment

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

LGTM!

z-a-f pushed a commit that referenced this pull request Oct 29, 2019
Better error message for quantized dispatch
0c81f8a 
Fixes #28518

ghstack-source-id: 6136b4d
Pull Request resolved: #28635
zdevito pushed a commit to zdevito/ATen that referenced this pull request Oct 31, 2019
@facebook-github-bot
Better error message for quantized dispatch (#28635)
5e48476 
Summary:
Pull Request resolved: pytorch/pytorch#28635

Fixes #28518

Test Plan: Imported from OSS

Differential Revision: D18132566

Pulled By: z-a-f

fbshipit-source-id: 08acc3033b12a0b79b43a5346b7af100416ffa94
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Oct 31, 2019

@z-a-f merged this pull request in 28be2d4.

@smessmer
Copy link
Contributor

smessmer commented Nov 1, 2019

I don't think we want this. The core dispatch logic is agnostic of quantization and should stay this way. Quantization depends on dispatch, not the other way round.

@smessmer smessmer mentioned this pull request Nov 1, 2019
Closed
@z-a-f z-a-f mentioned this pull request Nov 2, 2019
Closed
@facebook-github-bot facebook-github-bot deleted the gh/zafartahirov/64/head branch November 3, 2019 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jianyuh jianyuh jianyuh approved these changes

@jamesr66a jamesr66a Awaiting requested review from jamesr66a

@jerryzh168 jerryzh168 Awaiting requested review from jerryzh168

Assignees

No one assigned

Labels

Merged oncall: quantization Quantization support in PyTorch

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@z-a-f @facebook-github-bot @smessmer @jianyuh @jamesr66a @jerryzh168 @mruberry