-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
[Feature] Support MinkowskiEngine with MinkResNet #1422
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #1422 +/- ##
==========================================
- Coverage 51.17% 50.89% -0.28%
==========================================
Files 212 217 +5
Lines 18284 18567 +283
Branches 2980 3031 +51
==========================================
+ Hits 9356 9450 +94
- Misses 8371 8549 +178
- Partials 557 568 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Once one comment left fixed, the PR is ready to merge. |
Looks like there is no |
yes, sorry it's my mistake, just import from |
Hi @filaPro , |
Hi @ZwwWayne, apt-get install -y python3-dev libopenblas-dev
pip install ninja
pip install -U git+https://github.com/NVIDIA/MinkowskiEngine --install-option="--blas=openblas" -v --no-deps with conda install openblas-devel -c anaconda
pip install -U git+https://github.com/NVIDIA/MinkowskiEngine -v --no-deps --install-option="--blas_include_dirs=/opt/conda/include" --install-option="--blas=openblas" Is it ok? Also don't quite understand which of these 2 lines can we add to |
I think we can simply use these two lines. |
Motivation
Support MinkowskiEngine as a part of FCAF3D development plan. This sparse convolution backend can be used for other detection or segmentation methods.
Modification
Add docs about MinkowskiEngine installation and MinkResNet model with test.
BC-breaking (Optional)
No.