Skip to content

numeric suite: add types to eager#51168

Closed
vkuzo wants to merge 1 commit intogh/vkuzo/215/basefrom
gh/vkuzo/215/head
Closed

numeric suite: add types to eager#51168
vkuzo wants to merge 1 commit intogh/vkuzo/215/basefrom
gh/vkuzo/215/head

Conversation

@vkuzo
Copy link
Copy Markdown
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
Copy Markdown
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: 4441a69
Pull Request resolved: #51168
@vkuzo vkuzo requested review from hx89 and jerryzh168 January 27, 2021 01:17
@codecov
Copy link
Copy Markdown

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
Copy Markdown
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
laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
Pull Request resolved: pytorch#51168

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

Test Plan:
```
mypy torch/quantization/
```

Imported from OSS

Reviewed By: jerryzh168

Differential Revision: D26092454

fbshipit-source-id: d37cf61e4d9604f4bc550b392f55fb59165f7624
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.

3 participants