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

numeric suite: add types to eager #51168

Closed
wants to merge 1 commit into from
Closed

Conversation

vkuzo
Copy link
Contributor

@vkuzo vkuzo commented Jan 27, 2021

Stack from ghstack:

Summary:

Adds types to function I/O for numeric suite. This is for readability
and static type checking with mypy.

Test Plan:

mypy torch/quantization/

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D26092454

Summary:

Adds types to function I/O for numeric suite.  This is for readability
and static type checking with mypy.

Test Plan:

```
mypy torch/quantization/
```

Reviewers:

Subscribers:

Tasks:

Tags:

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

facebook-github-bot commented Jan 27, 2021

💊 CI failures summary and remediations

As of commit 416ca5e (more details on the Dr. CI page):


  • 1/1 failures possibly* introduced in this PR
    • 1/1 non-CircleCI failure(s)

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 to the (internal) Dr. CI Users group.

vkuzo added a commit that referenced this pull request Jan 27, 2021
Summary:

Adds types to function I/O for numeric suite.  This is for readability
and static type checking with mypy.

Test Plan:

```
mypy torch/quantization/
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: 4441a698bda34be3acc4648b017673551ab18d0d
Pull Request resolved: #51168
@vkuzo vkuzo requested review from hx89 and jerryzh168 January 27, 2021 01:17
@codecov
Copy link

codecov bot commented Jan 27, 2021

Codecov Report

Merging #51168 (416ca5e) into gh/vkuzo/215/base (345844d) will decrease coverage by 0.34%.
The diff coverage is 100.00%.

@@                  Coverage Diff                  @@
##           gh/vkuzo/215/base   #51168      +/-   ##
=====================================================
- Coverage              80.87%   80.53%   -0.35%     
=====================================================
  Files                   1931     1931              
  Lines                 210547   210548       +1     
=====================================================
- Hits                  170278   169555     -723     
- Misses                 40269    40993     +724     

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in e9ffad0.

@facebook-github-bot facebook-github-bot deleted the gh/vkuzo/215/head branch January 31, 2021 15:18
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