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] ReflectionPad2d #48036

Closed
wants to merge 5 commits into from
Closed

[quant] ReflectionPad2d #48036

wants to merge 5 commits into from

Conversation

z-a-f
Copy link
Contributor

@z-a-f z-a-f commented Nov 16, 2020

Stack from ghstack:

Differential Revision: D25000347

[ghstack-poisoned]
@dr-ci
Copy link

dr-ci bot commented Nov 16, 2020

💊 CI failures summary and remediations

As of commit 7cc6b2e (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 13 times.

pad_l, pad_t);
});
if (input.is_quantized()) {
AT_DISPATCH_QINT_TYPES(input.scalar_type(), "qreflection_pad2d", [&] {
Copy link
Contributor

Choose a reason for hiding this comment

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

should we just add a dispatch that works for both floating types and qint types?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done!

Copy link
Contributor

@jerryzh168 jerryzh168 left a comment

Choose a reason for hiding this comment

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

looks good

@codecov
Copy link

codecov bot commented Nov 17, 2020

Codecov Report

Merging #48036 (7cc6b2e) into gh/z-a-f/85/base (013e6a3) will decrease coverage by 0.00%.
The diff coverage is 66.66%.

@@                 Coverage Diff                  @@
##           gh/z-a-f/85/base   #48036      +/-   ##
====================================================
- Coverage             81.30%   81.29%   -0.01%     
====================================================
  Files                  1839     1839              
  Lines                198446   198452       +6     
====================================================
- Hits                 161337   161332       -5     
- Misses                37109    37120      +11     

@facebook-github-bot
Copy link
Contributor

@z-a-f merged this pull request in e1a1016.

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