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

replacing dtype torch.uint8 with torch.bool for indexing in pytorch 1.2.0 #1126

Merged
merged 1 commit into from
Oct 16, 2019

Conversation

garycao-cv
Copy link
Contributor

@garycao-cv garycao-cv commented Oct 7, 2019

Indexing with dtype torch.uint8 is deprecated in pytorch 1.2.0. Current version generates lots of warnings like "/opt/conda/conda-bld/pytorch_1565272271120/work/aten/src/ATen/native/IndexingUtils.h:20: UserWarning: indexing with dtype torch.uint8 is now deprecated, please use a dtype torch.bool instead."
In this PR, dtype torch.uint8 is replaced with torch.bool for indexing in compliance with pytorch 1.2.0.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Oct 7, 2019
@botcs
Copy link
Contributor

botcs commented Oct 7, 2019

Hey @garycao-cv thanks for the PR
Could you please compare your version with #1093 ? There are some extra additions as well. If there are some differences please make a PR to that feature branch and I'll merge it.

Make sure all tests are passing and COCO R50 FPN scores can be reproduced, using the weight file from model zoo

@garycao-cv
Copy link
Contributor Author

garycao-cv commented Oct 7, 2019

Thanks @botcs.
All tests passed. When I tried to create a PR targeting your feature branch, I could not find your repo/feature branch in the target box. Is your branch open to PRs? or did I miss anything here?
Thanks.

@botcs
Copy link
Contributor

botcs commented Oct 8, 2019

Did you try it here? https://github.com/botcs/maskrcnn-benchmark/pulls

@botcs botcs added the good first issue Good for newcomers label Oct 8, 2019
@garycao-cv
Copy link
Contributor Author

@botcs Yes, I tried. Still could not target your branch for creating a PR. I'm attaching two screen shots here to show you what I saw on my side by following the link you shared. (PR-step2 is what I got after I clicked the "New Pull Request" in PR-step1)

PR-step1.pdf
PR-step2.pdf

@garycao-cv
Copy link
Contributor Author

@botcs any thoughts? Could you see if you can create a PR on your side to merge my changes into your branch? Thanks.

@botcs
Copy link
Contributor

botcs commented Oct 16, 2019

ummm. on the second image, botcs is on the list...

So please try again on this link: botcs/maskrcnn-benchmark@pytorch-v1.2...botcs:pytorch-v1.2 but please select your head branch (the box on the right)

@garycao-cv
Copy link
Contributor Author

garycao-cv commented Oct 16, 2019

@botcs thanks for the response. But I'm quite confused by it. On the second image, there are five repos on the list after I type bo, which are facebookresearch, Bowenbao, Designbook1, aabobakr, Imsreborn. Is there an alias for botcs or I'm really missing something here. Thanks again for looking into this.

@garycao-cv
Copy link
Contributor Author

@botcs I can see botcs on the list of the right (i.e. source repo), but not on the list of the left (i.e. target repo). That's why I thought there may be some special setting on your side. Can you see your repo in the list of the left box?

@botcs botcs merged commit 454bd1d into facebookresearch:master Oct 16, 2019
@botcs
Copy link
Contributor

botcs commented Oct 16, 2019

closing my PR

@garycao-cv
Copy link
Contributor Author

@botcs thanks for taking care of this!

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. good first issue Good for newcomers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants