Skip to content
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.

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

[quant][graphmode][fx] Add support for qat convbn{relu}1d #47248

Closed
wants to merge 9 commits into from

Conversation

jerryzh168
Copy link
Contributor

@jerryzh168 jerryzh168 commented Nov 3, 2020

Stack from ghstack:

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D24696524

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
@jerryzh168 jerryzh168 requested a review from vkuzo November 3, 2020 01:17
Copy link
Contributor

@vkuzo vkuzo left a comment

Choose a reason for hiding this comment

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

lg, some optional comments inline. Should we also add a numerical correctness test?

torch/nn/quantized/modules/conv.py Outdated Show resolved Hide resolved
@jerryzh168
Copy link
Contributor Author

ok, I think we can change the test to test numerics too

@dr-ci
Copy link

dr-ci bot commented Nov 3, 2020

💊 CI failures summary and remediations

As of commit 1afb939 (more details on the Dr. CI page):


💚 💚 Looks good so far! There are no failures yet. 💚 💚


This comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.

Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group.

See how this bot performed.

This comment has been revised 30 times.

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D24696524](https://our.internmc.facebook.com/intern/diff/D24696524)

[ghstack-poisoned]
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D24696524](https://our.internmc.facebook.com/intern/diff/D24696524)

[ghstack-poisoned]
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D24696524](https://our.internmc.facebook.com/intern/diff/D24696524)

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Nov 3, 2020
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: d5390f4890c1d35de7af5828df23cf365fd4e452
Pull Request resolved: #47248
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D24696524](https://our.internmc.facebook.com/intern/diff/D24696524)

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Nov 3, 2020
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 42c0043f3593b21fbd33ce36f2e3916fa9d4e8a7
Pull Request resolved: #47248
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D24696524](https://our.internmc.facebook.com/intern/diff/D24696524)

[ghstack-poisoned]
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D24696524](https://our.internmc.facebook.com/intern/diff/D24696524)

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Nov 3, 2020
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 11f606055264063f2cd7ef4baff18e3ca6b50995
Pull Request resolved: #47248
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: [D24696524](https://our.internmc.facebook.com/intern/diff/D24696524)

[ghstack-poisoned]
jerryzh168 added a commit that referenced this pull request Nov 4, 2020
Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 4468f1f185227675477a3207e8b82c2a9967151a
Pull Request resolved: #47248
@codecov
Copy link

codecov bot commented Nov 4, 2020

Codecov Report

Merging #47248 into gh/jerryzh168/487/base will increase coverage by 0.00%.
The diff coverage is 98.57%.

@@                   Coverage Diff                   @@
##           gh/jerryzh168/487/base   #47248   +/-   ##
=======================================================
  Coverage                   60.81%   60.82%           
=======================================================
  Files                        2749     2749           
  Lines                      254098   254126   +28     
=======================================================
+ Hits                       154536   154562   +26     
- Misses                      99562    99564    +2     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 0cba3e3.

@facebook-github-bot facebook-github-bot deleted the gh/jerryzh168/487/head branch November 7, 2020 15:17
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.

None yet

3 participants