Skip to content

Add type annotations to torch.overrides#50824

Closed
guilhermeleobas wants to merge 22 commits intopytorch:masterfrom
guilhermeleobas:override
Closed

Add type annotations to torch.overrides#50824
guilhermeleobas wants to merge 22 commits intopytorch:masterfrom
guilhermeleobas:override

Conversation

@guilhermeleobas
Copy link
Copy Markdown
Collaborator

This is a follow up PR of #48493.

Fixes #48492

@guilhermeleobas guilhermeleobas added the module: typing Related to mypy type annotations label Jan 20, 2021
@guilhermeleobas guilhermeleobas self-assigned this Jan 20, 2021
@ngimel ngimel requested a review from hameerabbasi January 21, 2021 05:01
@ngimel ngimel added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Jan 21, 2021
Copy link
Copy Markdown
Collaborator

@hameerabbasi hameerabbasi left a comment

Choose a reason for hiding this comment

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

One minor comment.

Comment thread torch/overrides.py Outdated
guilhermeleobas and others added 2 commits January 21, 2021 12:22
Co-authored-by: Hameer Abbasi <einstein.edison@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 21, 2021

Codecov Report

Merging #50824 (82d576e) into master (7f3a407) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #50824      +/-   ##
==========================================
- Coverage   80.66%   80.65%   -0.01%     
==========================================
  Files        1913     1913              
  Lines      208058   208059       +1     
==========================================
- Hits       167833   167819      -14     
- Misses      40225    40240      +15     

Copy link
Copy Markdown
Collaborator

@hameerabbasi hameerabbasi left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM.

Copy link
Copy Markdown
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@ezyang has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Copy Markdown
Contributor

@ezyang merged this pull request in 9dfbfe9.

laurentdupin pushed a commit to laurentdupin/pytorch that referenced this pull request Apr 24, 2026
Summary:
This is a follow up PR of pytorch#48493.

Fixes pytorch#48492

Pull Request resolved: pytorch#50824

Reviewed By: bdhirsh

Differential Revision: D26050736

Pulled By: ezyang

fbshipit-source-id: 049605fd271cff28c8b6e300c163e9df3b3ea23b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla signed Merged module: typing Related to mypy type annotations open source triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enable torch.overrides typechecks during CI

5 participants