Skip to content

Conversation

zhiqwang
Copy link
Contributor

@zhiqwang zhiqwang commented Dec 15, 2020

Hi , this PR is to Fix #3172 .

@zhiqwang
Copy link
Contributor Author

zhiqwang commented Dec 15, 2020

There are some BroadcastingList2 like

from torch.jit.annotations import List, BroadcastingList2

left without replacement, for I'm not sure whether it can be replaced.

Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

Looks great, thanks!

The BroadcastingList2 needs to be kept from torch.jit.annotations as you did.

@fmassa fmassa merged commit 90645cc into pytorch:master Dec 15, 2020
facebook-github-bot pushed a commit that referenced this pull request Dec 23, 2020
Summary:
* Replacing all torch.jit.annotations with typing

* Replacing remaining typing

Reviewed By: fmassa

Differential Revision: D25679213

fbshipit-source-id: 297d52d7ed1322d350619e298a9c2bbaa771d2a2
@zhiqwang zhiqwang deleted the replacing_typing_annotations branch May 8, 2021 22:29
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.

Replace all torch.jit.annotations with typing annotations
3 participants