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

FastSurfer surf pipeline did not finish: Missing .label files (not enough memory for mris_sample_parc) #503

Closed
mwendhaus opened this issue Apr 5, 2024 · 11 comments
Labels
question Further information is requested

Comments

@mwendhaus
Copy link

mwendhaus commented Apr 5, 2024

Hello,
I am running FastSurfer using the provided Docker container from an Intel MacBook Pro. I use this command:

#!/bin/bash

export FREESURFER_HOME=/Applications/freesurfer/7.4.1
source $FREESURFER_HOME/SetUpFreeSurfer.sh

docker run            -v .../nifti_files/:/data \
                      -v ...:/output \
                      -v ...:/fs_license \
                      --rm --user $(id -u):$(id -g) deepmi/fastsurfer:latest \
                      --fs_license /fs_license/license.txt \
                      --t1 /data/mri_data_t1.nii.gz \
                      --sid subjectX --sd /output \
                      --parallel \
                      --3T \

The code runs fine, however I am missing several .label files in the resulting label folder, which are described in the OUTPUT_FILES.md, such as lh.aparc.DKTatlas.mapped.annot. The log files do not suggest any obvious error. I am thankful for any hint on what I might be doing wrong!
image

@mwendhaus mwendhaus added the question Further information is requested label Apr 5, 2024
@dkuegler
Copy link
Member

dkuegler commented Apr 5, 2024

Hi,

first off, you dont need to source FreeSurfer if you are using the docker container.

As for the data: Can you share your log files anyways, so we can have a look?

Kind regards,
David

@mwendhaus
Copy link
Author

Thank you very much for the fast reply. I am attaching the recon-all.log, if any other file might be necessary please let me know.
recon-all.log

@dkuegler
Copy link
Member

dkuegler commented Apr 5, 2024

Sorry, can you please send the deep-seg.log and the recon-surf.log?

@mwendhaus
Copy link
Author

Of course:
deep-seg.log
recon-surf.log

@dkuegler
Copy link
Member

dkuegler commented Apr 8, 2024

mris_sample_parc gets terminated
Command terminated by signal 9

We have had some issues with mris_sample_parc acquiring excessive amounts of memory. This issue has been fixed in #430 , but we have not released a new docker image yet.
You can try giving docker more memory by for example --shm-size 8G see https://docs.docker.com/engine/reference/run/#runtime-constraints-on-resources and see also https://github.com/Deep-MI/FastSurfer/tree/dev/Docker#example-3-experimental-build-for-amd-gpus

@mwendhaus
Copy link
Author

Great, thank you very much for the help!

@mwendhaus
Copy link
Author

Just to complete this case, I got the same error using --shm-size 8G, and increased to --shm-size 12G to make sure. This worked fine and all necessary files were generated.

@dkuegler
Copy link
Member

@mwendhaus I am confused, you got the same error, BUT all necessary files were generated?

@mwendhaus
Copy link
Author

I got the same error as before when I used the flag --shm-size 8G. However, when I further increased the memory to 12G, I did not get the error, and the files were created.

@dkuegler
Copy link
Member

Thank you! I believe this underlying issue is solved with the current dev version anyways, but good you were able to run FastSurfer.

@dkuegler dkuegler changed the title Missing .label files FastSurfer surf pipeline did not finish: Missing .label files (not enough memory for mris_sample_parc) Apr 11, 2024
@mwendhaus
Copy link
Author

Thank you for the help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants