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

fx quant: more typehints, part 3 #48794

Closed
wants to merge 1 commit into from
Closed

Conversation

vkuzo
Copy link
Contributor

@vkuzo vkuzo commented Dec 3, 2020

Stack from ghstack:

Summary:

Adds typehints to function I/O in torch/quantization/quantize_fx.py,
for readability.

Test Plan:

mypy torch/quantization/

Reviewers:

Subscribers:

Tasks:

Tags:

Differential Revision: D25307084

Summary:

Adds typehints to function I/O in `torch/quantization/quantize_fx.py`,
for readability.

Test Plan:

```
mypy torch/quantization/
```

Reviewers:

Subscribers:

Tasks:

Tags:

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

Adds typehints to function I/O in `torch/quantization/quantize_fx.py`,
for readability.

Test Plan:

```
mypy torch/quantization/
```

Reviewers:

Subscribers:

Tasks:

Tags:

ghstack-source-id: daad7f141e85bc2c8d9ed415f19094c3a444fa7a
Pull Request resolved: #48794
@codecov
Copy link

codecov bot commented Dec 3, 2020

Codecov Report

Merging #48794 (2ca75cd) into gh/vkuzo/178/base (b9ef984) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@                Coverage Diff                 @@
##           gh/vkuzo/178/base   #48794   +/-   ##
==================================================
  Coverage              80.73%   80.73%           
==================================================
  Files                   1863     1863           
  Lines                 200897   200898    +1     
==================================================
+ Hits                  162185   162186    +1     
  Misses                 38712    38712           

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in f5d9424.

@facebook-github-bot facebook-github-bot deleted the gh/vkuzo/178/head branch December 7, 2020 15:21
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