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

denseCRF3D import error #605

Closed
rijobro opened this issue Feb 3, 2022 · 6 comments · Fixed by #606
Closed

denseCRF3D import error #605

rijobro opened this issue Feb 3, 2022 · 6 comments · Fixed by #606
Assignees
Labels
0.4.0 Targeted to Release version 0.4 bug Something isn't working

Comments

@rijobro
Copy link
Contributor

rijobro commented Feb 3, 2022

In a clean conda environment, I did the following to install MONAILabel (wanted it from source so I could use my branch)

conda create -n monai-label python=3.9 -y
conda activate monai-label
conda install pytorch cudatoolkit=11.3 -c pytorch -y
cd <path/to/MONAILabel>
pip install -e .

Then the following line causes an error

python -c "monailabel start_server ..."

And more specifically, so does the following

python -c "from monailabel.scribbles.transforms import denseCRF3D"

Both times the error is as follows:

Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/rich/Documents/Code/MONAILabel/monailabel/scribbles/transforms.py", line 16, in <module>
    import denseCRF3D
ImportError: /home/rich/miniconda3/envs/monai-label/lib/python3.9/site-packages/denseCRF3D.cpython-39-x86_64-linux-gnu.so: undefined symbol: _ZSt28__throw_bad_array_new_lengthv

Any idea what's going on?

@SachidanandAlle
Copy link
Collaborator

@masadcv can you help and unblock @rijobro

@SachidanandAlle
Copy link
Collaborator

@SachidanandAlle
Copy link
Collaborator

@masadcv can you handle the import error and automatically exclude scribbles in the app if import fails due to xyz reasons..

@SachidanandAlle SachidanandAlle added the 0.4.0 Targeted to Release version 0.4 label Feb 3, 2022
@diazandr3s
Copy link
Collaborator

Thanks, @rijobro for reporting this.
I agree with @SachidanandAlle. Please comment those lines out while this issue is solved.
There is another conversation around simpleCRF here.
Let's see how this progress :)

@masadcv
Copy link
Collaborator

masadcv commented Feb 3, 2022

Thanks @SachidanandAlle and @diazandr3s !
I am looking into this.

@rijobro this error is coming from SImpleCRF installation - I am unsure why it is causing this. As Andres mentioned, there is another ticket to resolve some installation issues with SimpleCRF on windows.

For now, the easiest and quickest solution would be to make these optional_import (unless I am missing something). This means you will lose Scribbles functionality, but be able to get around the errors and use everything else until we find a fix. I will add a PR to do this for all SimpleCRF imports

@rijobro
Copy link
Contributor Author

rijobro commented Feb 4, 2022

Ok all will disable Scribbles for the time being, thanks for the help!

@rijobro rijobro added the bug Something isn't working label Feb 4, 2022
@SachidanandAlle SachidanandAlle linked a pull request Feb 4, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.4.0 Targeted to Release version 0.4 bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants