-
Notifications
You must be signed in to change notification settings - Fork 28
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
BUG: list index out of range if image.direction is not (1,0,0,0,-1,0,0,0,1) #135
Comments
Hi @puccj, many thanks for the elaborate bug and also for finding your way around the bug - very helpful :)! We will look into the issue. I have a feeling about why it failed, might be mainly because of the orientation. But let me be sure. @Keyn34 @mprires: would you be able to take a crack at it? I am gone till next week. |
Hello, I am having the same issue (list index out of range) when using both the cli tool and the Python API. |
It is correct that nnUnet is not producing any results, and I am currently looking into that. I could not replicate the error for myself for now, but I am trying to determine the differences. As of now, it is likely that the IOFactory of SimpleITK can't handle the direction of the images correctly, as @puccj pointed out. I will let you know ASAP! |
@mordilos in your case, I think it might be the env variables. Would you be kind enough to let me know about the environment and relevant OS/hardware details so that we can help you better? Lalith |
@LalithShiyam yes of course. |
@mordilos its definitely the environment variables. Make sure you source the file and test if you can see the env variables before running moose. keep me posted :) |
@LalithShiyam ok so I think it's not just the env variables.
inside the nnunet_trained_models: Dataset123_Organs/nnUNetTrainer_2000epochs_NoMirroring__nnUNetPlans__3d_fullres/ the files that are automatically downloaded but still, nnunet won't produce the outputs. |
@mordilos strange. We don't have the same setup you have and it is hard for us to test it. Have you tried it without docker/jupyterhub setup? Just asking to figure out if you have the same error. |
@LalithShiyam I know, I will try to reproduce it on a native linux machine. I will keep you posted. |
Sorry about not being helpful - keep me posted and we will figure this out @mordilos :) |
Hi, first of all thank you for this wonderful software. I was trying to use it on the VerSe database, specifically on this version here. I have tried to run moosez in:
In both cases, I worked in a conda environment with python 3.10 and the latest version of moosez installed with pip.
When I use the command
moosez -d <direcory> -m clin_ct_vertebrae
, it results in three different errors for some of the subjects:The same error always arises from the same subject. The last two errors occur instantly as (or soon after) moose is run and their outputs are:
⠧ [1/52] Running prediction for sub-verse092 using clin_ct_vertebrae...Killed
The first error is where it gets interesting. The error does not occur during the prediction, but in the postprocess operations, as you can understand both from the fact that the error is thrown after some time or just by its output:
Since the single-line execution of moose is interrupted in all of the three cases I couldn't use some sort of error management and I manually re-run moose many times to differentiate between the "good" and "bad" subjects. I found that moose works for only 62 of them out of the total 160. I then used Simple ITK to see if there were some common features that differentiate the good and the bad images. What I found was that:
In both cases there are exceptions: images that have the "right" direction and/or spacing but for which moose execution fails
I tried to set the direction to (1, 0, 0 ,0 , -1, 0, 0, 0, 1), using SimpleITK and moose worked! At least for the majority of the images for which moose used to not work, since there are still some for which the three errors arises (I've seen all three of them)
I'm still currently running the predictions (again, manually sorting the "new good" and "new bad" subjects), so I don't know the precise number of the new good and bad subjects, but at least it worked.
To Reproduce
Steps to reproduce the behavior:
conda create -n moosez python=3.10
pip install moosez
moose -d <folder_path> -m clin_ct_vertebrae
and wait for it to failConclusion
I want to point out that I don't really need you to solve this issue: I just used moose for a little university exam and I think I won't use it anymore (just because I will finish my degree, not because something is wrong with the software). I wrote this issue only because I think it will be usefull for you in order to fix a bug, and for other if they encounter the same problem. For the same reason, I hope you will pardon me if I won't reply in short time to your answer (that I'm sure will come soon). I mean, for what it concerns me, you could close this issue straight away.
The text was updated successfully, but these errors were encountered: