Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix mask.py #576

Open
wants to merge 1 commit into
base: pytorch-1.0
Choose a base branch
from
Open

fix mask.py #576

wants to merge 1 commit into from

Conversation

kentaroy47
Copy link

fix for #410

will work if you have cocoapi installed.

@kentaroy47
Copy link
Author

similar to unmerged PR in #466, but there were a lot of unnecessary changes there.

@EMCP
Copy link
Contributor

EMCP commented Jul 16, 2019

this actually did not work... with a blank fresh copy of the project I am seeing

(pytorch1_py37_tut) emcp@emcp-corei7-2600k:/media/latest_mount/Dev/git/jwyang/faster-rcnn.pytorch$ vim lib/pycocotools/mask.py 
(pytorch1_py37_tut) emcp@emcp-corei7-2600k:/media/latest_mount/Dev/git/jwyang/faster-rcnn.pytorch$ python trainval_net.py --net res101 --dataset pascal_voc --cuda --bs 2 
Traceback (most recent call last):
  File "trainval_net.py", line 27, in <module>
    from roi_data_layer.roidb import combined_roidb
  File "/media/latest_mount/Dev/git/jwyang/faster-rcnn.pytorch/lib/roi_data_layer/roidb.py", line 9, in <module>
    from datasets.factory import get_imdb
  File "/media/latest_mount/Dev/git/jwyang/faster-rcnn.pytorch/lib/datasets/factory.py", line 15, in <module>
    from datasets.coco import coco
  File "/media/latest_mount/Dev/git/jwyang/faster-rcnn.pytorch/lib/datasets/coco.py", line 23, in <module>
    from pycocotools.coco import COCO
  File "/media/latest_mount/Dev/git/jwyang/faster-rcnn.pytorch/lib/pycocotools/coco.py", line 60, in <module>
    from . import mask
  File "/media/latest_mount/Dev/git/jwyang/faster-rcnn.pytorch/lib/pycocotools/mask.py", line 3, in <module>
    from pycocotools._mask import _mask
ModuleNotFoundError: No module named 'pycocotools._mask'

@EMCP
Copy link
Contributor

EMCP commented Jul 16, 2019

similar to unmerged PR in #466, but there were a lot of unnecessary changes there.

those updates brought the latest version into the project.. and it worked when I performed it.

@kentaroy47
Copy link
Author

@EMCP
Thanks for pointing this out.
Yes, i found it doesn't work.. let me look into this.

(another quick workaround is commenting out import mask in the script if you're just doing binding box detection..)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants