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

The installation of mmrotate will install an additional mmcv. #6

Closed
liuyanyi opened this issue Feb 18, 2022 · 7 comments
Closed

The installation of mmrotate will install an additional mmcv. #6

liuyanyi opened this issue Feb 18, 2022 · 7 comments

Comments

@liuyanyi
Copy link
Collaborator

Describe the bug
The installation of mmrotate will install an additional mmcv although I have mmcv-full installed.

The output of the pip list command is as follows:

mmcv                1.4.5
mmcv-full           1.4.5
mmdet               2.21.0
mmrotate            0.1.0     

Reproduction

  1. What command or script did you run?
pip install -v -e .

Bug fix

Compared with other mmlab project, i think the difference may appeare in requirements/runtime.txt, maybe remove mmcv in runtime.txt can fix it.

@zytx121
Copy link
Collaborator

zytx121 commented Feb 18, 2022

This is because only the latest version of mmcv (1.4.5) contains all the operators required by MMRotate.

@liuyanyi
Copy link
Collaborator Author

Thanks for your reply, I follow the installation guide in a new conda environment, there are still two mmcv in pip list. Both training and testing work fine.
But on mmcv's document, it says both versions in the same environment may encounter errors, should I uninstall one of them?

@liufeinuaa
Copy link
Contributor

I also have this problem, bug training and working fine....

@liufeinuaa
Copy link
Contributor

liufeinuaa commented Feb 20, 2022

I was tried modified the requirements/runtime.txt, change the mmcv to mmcv-full, can fix this problem.
In my conda env just one mmcv-full package, and run demo was fine.

@ZwwWayne
Copy link
Collaborator

Hi @liufeinuaa ,
Thanks for your solution. Would you like to create a PR to fix that?

@liufeinuaa
Copy link
Contributor

Of course

Hi @liufeinuaa , Thanks for your solution. Would you like to create a PR to fix that?

@ZwwWayne
Copy link
Collaborator

resolved in #17

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

No branches or pull requests

4 participants