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

KeyError: 'Cascade Mask R-CNN' #125

Closed
tik0 opened this issue Jun 3, 2022 · 7 comments
Closed

KeyError: 'Cascade Mask R-CNN' #125

tik0 opened this issue Jun 3, 2022 · 7 comments

Comments

@tik0
Copy link

tik0 commented Jun 3, 2022

  1. I've followed the installation steps from https://mmdetection.readthedocs.io/en/v2.25.0/get_started.html
  2. Then I try to execute the following command from the manual mim download mmdet --config yolov3_mobilenetv2_320_300e_coco --dest .
$ mim download mmdet --config yolov3_mobilenetv2_320_300e_coco --dest .
~/miniconda3/envs/openmmlab/lib/python3.8/site-packages/_distutils_hack/__init__.py:30: UserWarning: Setuptools is replacing distutils.
  warnings.warn("Setuptools is replacing distutils.")
Traceback (most recent call last):
  File "~/miniconda3/envs/openmmlab/bin/mim", line 33, in <module>
    sys.exit(load_entry_point('openmim', 'console_scripts', 'mim')())
  File "~/miniconda3/envs/openmmlab/lib/python3.8/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "~/miniconda3/envs/openmmlab/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "~/miniconda3/envs/openmmlab/lib/python3.8/site-packages/click/core.py", line 1259, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "~/miniconda3/envs/openmmlab/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "~/miniconda3/envs/openmmlab/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "~/repositories/mim/mim/commands/download.py", line 44, in cli
    download(package, configs, dest_root)
  File "~/repositories/mim/mim/commands/download.py", line 75, in download
    model_info = get_model_info(
  File "~/repositories/mim/mim/commands/search.py", line 170, in get_model_info
    dataframe = convert2df(metadata)
  File "~/repositories/mim/mim/commands/search.py", line 396, in convert2df
    for key, value in name2collection[collection_name].items():
KeyError: 'Cascade Mask R-CNN'

I've also tried to install mim directly from master, but the same error appears.

@ice-tong
Copy link
Collaborator

ice-tong commented Jun 3, 2022

Hi @tik0 , this looks like a bug introduced in mmdet==2.25.0, related commit: open-mmlab/mmdetection@1fd48f7.

A quick workaround is to use mmdet<2.25.0.

@ice-tong
Copy link
Collaborator

ice-tong commented Jun 3, 2022

The same issue under mmdet: open-mmlab/mmdetection#8122

@tamersalama
Copy link

So - how would I specify mmdet<2.25.0 during download? I see that the download command rejects the version number as part of the package name (it rejects mmdet==2.24.0 for example).

@ice-tong
Copy link
Collaborator

Hi @tamersalama , you should install mmdet<2.25.0. For example: mim install mmdet==2.24.0.
In addition, if you are interested, you can create a PR for mmdet to fix this problem, just change the Cascade Mask R-CNN in this file to Mask R-CNN.

@br3nr
Copy link

br3nr commented Jun 16, 2022

@ice-tong Hello, I am happy to do this, however I am unsure how I would actually test these changes if you wouldn't mind advising :)

@ice-tong
Copy link
Collaborator

@RepetitiveRoutine Hi, you just need to modify two places in this file, change Cascade Mask R-CNN to Mask R-CNN, the steps in this tutorial will work fine.
The contributing doc can be found here. If you have any questions, feel free to feedback to us.

@ZwwWayne
Copy link
Collaborator

ZwwWayne commented Jun 25, 2022

Fixed in open-mmlab/mmdetection#8240

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 a pull request may close this issue.

5 participants