Skip to content
This repository has been archived by the owner on Oct 31, 2023. It is now read-only.

Rollback dispatch patch #603

Merged
merged 5 commits into from
Mar 26, 2019

Conversation

miguelvr
Copy link
Contributor

Reverts AT_DISPATCH_FLOATING_TYPES patch, which lead to a compilation error with the latest version of pytorch and pytorch-nightly

FIxes #601

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Mar 26, 2019
@miguelvr
Copy link
Contributor Author

cc @fmassa

@fmassa
Copy link
Contributor

fmassa commented Mar 26, 2019

cc @vishwakftw I believe this can be reverted now, given that there was a backward-compatibility fix sent to PyTorch?

@fmassa
Copy link
Contributor

fmassa commented Mar 26, 2019

Also note that this will be introducing deprecation warnings during compilation time pytorch/pytorch#17996

Copy link
Contributor

@vishwakftw vishwakftw left a comment

Choose a reason for hiding this comment

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

Thank you for fixing this. Looks good to me.

Copy link
Contributor

@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.

Thanks!

@fmassa fmassa merged commit 4a7dcc4 into facebookresearch:master Mar 26, 2019
eskjorg pushed a commit to eskjorg/maskrcnn-benchmark that referenced this pull request Mar 27, 2019
* Merge branch 'master' of /home/braincreator/projects/maskrcnn-benchmark with conflicts.

* rolls back the breaking AT dispatch changes (facebookresearch#555)

* revert accidental docker changes

* revert accidental docker changes (2)
Lyears pushed a commit to Lyears/maskrcnn-benchmark that referenced this pull request Jun 28, 2020
* Merge branch 'master' of /home/braincreator/projects/maskrcnn-benchmark with conflicts.

* rolls back the breaking AT dispatch changes (facebookresearch#555)

* revert accidental docker changes

* revert accidental docker changes (2)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed Do not delete this pull request or issue due to inactivity.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants