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

Merge with main mmdetection3d repo? #13

Closed
stanleyshly opened this issue Dec 11, 2021 · 2 comments
Closed

Merge with main mmdetection3d repo? #13

stanleyshly opened this issue Dec 11, 2021 · 2 comments

Comments

@stanleyshly
Copy link

Given this work is modified based on the mmdetection3d repo, are there any plans to update this code to the original mmdetection3d repo?

@filaPro
Copy link
Contributor

filaPro commented Dec 11, 2021

Hi @stanleyshly,

There is one problem on this way: dependencies of MinkowskiEngine and RotatedIoU. Both of them require CUDA builds and MinkowskiEngine builds for several minutes during installation. I'm not sure if authors of mmdetection3d will welcome these dependencies.

Probably, the convolutional backbone and neck can be rewritten with spconv, supported in mmdetection3d but I'm not ready to analyse it and probably it would be slower than with MinkowskiEngine.

@stanleyshly
Copy link
Author

Thank you your your prompt response, and I do apologize for the slow reply. Thank you for the information, it makes sense, I'll stick with this repo for now.

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

2 participants