Skip to content

Conversation

yhcharles
Copy link
Contributor

Summary:
Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.

Differential Revision: D41028360

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Nov 8, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D41028360

yhcharles added a commit to yhcharles/tnt that referenced this pull request Nov 8, 2022
Summary:
X-link: meta-pytorch/torchrec#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.

Differential Revision: D41028360

fbshipit-source-id: bb21a775019614e4ebcca9dc61872f05e7f3aa7a
yhcharles added a commit to yhcharles/torchrec that referenced this pull request Nov 9, 2022
Summary:
X-link: pytorch/pytorch#88701

X-link: meta-pytorch/tnt#269

X-link: facebookresearch/ReAgent#691

X-link: meta-pytorch/torchsnapshot#129

Pull Request resolved: meta-pytorch#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Differential Revision: D41028360

fbshipit-source-id: c272bcc28d2df9477329ed03d5e92b1544b156c4
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D41028360

yhcharles added a commit to yhcharles/pytorch that referenced this pull request Nov 9, 2022
Summary:
Pull Request resolved: pytorch#88701

X-link: meta-pytorch/tnt#269

X-link: facebookresearch/ReAgent#691

X-link: meta-pytorch/torchsnapshot#129

X-link: meta-pytorch/torchrec#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Test Plan: CI

Differential Revision: D41028360

fbshipit-source-id: 5a19e70a4cfd7070fcbe786d6a31b67a895b2c5b
Summary:
X-link: pytorch/pytorch#88701

X-link: meta-pytorch/tnt#269

X-link: facebookresearch/ReAgent#691

X-link: meta-pytorch/torchsnapshot#129

Pull Request resolved: meta-pytorch#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Differential Revision: D41028360

fbshipit-source-id: 6c010352a184790f58cd9730b35e68043de3ca2d
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D41028360

facebook-github-bot pushed a commit to meta-pytorch/tnt that referenced this pull request Nov 9, 2022
Summary:
X-link: pytorch/pytorch#88701

Pull Request resolved: #269

X-link: facebookresearch/ReAgent#691

X-link: meta-pytorch/torchsnapshot#129

X-link: meta-pytorch/torchrec#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Reviewed By: zhaojuanmao

Differential Revision: D41028360

fbshipit-source-id: 577f212ca4c47e23a8577c4b92385c483f96e2c1
facebook-github-bot pushed a commit to meta-pytorch/torchsnapshot that referenced this pull request Nov 9, 2022
Summary:
X-link: pytorch/pytorch#88701

X-link: meta-pytorch/tnt#269

X-link: facebookresearch/ReAgent#691

Pull Request resolved: #129

X-link: meta-pytorch/torchrec#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Reviewed By: zhaojuanmao

Differential Revision: D41028360

fbshipit-source-id: 577f212ca4c47e23a8577c4b92385c483f96e2c1
facebook-github-bot pushed a commit to facebookresearch/detectron2 that referenced this pull request Nov 9, 2022
Summary:
X-link: pytorch/pytorch#88701

X-link: meta-pytorch/tnt#269

X-link: facebookresearch/ReAgent#691

X-link: meta-pytorch/torchsnapshot#129

X-link: meta-pytorch/torchrec#799

Pull Request resolved: #4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Reviewed By: zhaojuanmao

Differential Revision: D41028360

fbshipit-source-id: 577f212ca4c47e23a8577c4b92385c483f96e2c1
facebook-github-bot pushed a commit to facebookresearch/ReAgent that referenced this pull request Nov 9, 2022
Summary:
X-link: pytorch/pytorch#88701

X-link: meta-pytorch/tnt#269

Pull Request resolved: #691

X-link: meta-pytorch/torchsnapshot#129

X-link: meta-pytorch/torchrec#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Reviewed By: zhaojuanmao

Differential Revision: D41028360

fbshipit-source-id: 577f212ca4c47e23a8577c4b92385c483f96e2c1
lequytra pushed a commit to lequytra/torchrec that referenced this pull request Dec 6, 2022
Summary:
X-link: pytorch/pytorch#88701

X-link: meta-pytorch/tnt#269

X-link: facebookresearch/ReAgent#691

X-link: meta-pytorch/torchsnapshot#129

Pull Request resolved: meta-pytorch#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Reviewed By: zhaojuanmao

Differential Revision: D41028360

fbshipit-source-id: 577f212ca4c47e23a8577c4b92385c483f96e2c1
xuruiyang pushed a commit to facebookresearch/ReAgent that referenced this pull request Sep 20, 2025
Summary:
X-link: pytorch/pytorch#88701

X-link: meta-pytorch/tnt#269

Pull Request resolved: #691

X-link: meta-pytorch/torchsnapshot#129

X-link: meta-pytorch/torchrec#799

X-link: facebookresearch/detectron2#4649

Context in https://fburl.com/4irjskbe

This change deletes distributed.pyi, so that lintrunner will run mypy on distributed.py for typing check.
It also helps to fix a lot pyre false alarms, so that we can remove those pyre suppressions.

Reviewed By: zhaojuanmao

Differential Revision: D41028360

fbshipit-source-id: 577f212ca4c47e23a8577c4b92385c483f96e2c1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants