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

23.1.0 release candidate 1 version throw errors about can not find precomputed segmentation while using --surface-recon-method mcribs #321

Closed
yanbin-niu opened this issue Nov 12, 2023 · 1 comment · Fixed by #322
Labels
bug Something isn't working

Comments

@yanbin-niu
Copy link

What happened?

Hi,

Today, we experimented with the latest version of nibabies (23.1.0rc1) to appreciate its performance in single anatomical processing. Unfortunately, we encountered an issue where the mcribs run was unable to locate the precomputed segmentation. Previously, when we used version 23.1.0rc0 with the --surface-recon-method mcribs flag and the --derivatives /opt/derivatives/precomputed, we do not have problems. Could you please assist us in resolving this issue with the new version?

Best regards,
Yanbin

What command did you use?

docker run --rm -e DOCKER_VERSION_8395080871=23.1.0rc1 -it ' \
                 '-v /Applications/freesurfer/7.1.1/license.txt:/opt/freesurfer/license.txt:ro ' \
                 '-v /Volumes/SEALab_Projects/BABIES/MRI/newborn/bids:/data:ro ' \
                 '-v /Volumes/SEALab_Projects/BABIES/MRI/newborn/derivatives/Nibabies:/out ' \
                 '-v /Volumes/SEALab_Projects/BABIES/MRI/newborn/derivatives/precomputed:/opt/derivatives/precomputed ' \
                 '-v /Volumes/SEALab_Projects/BABIES/MRI/newborn/derivatives/work/nibabies_work:/scratch ' \
                 'nipreps/nibabies:23.1.0rc1 /data /out participant ' \
                 f'--skip_bids_validation --age-months 1 --nprocs 6 --participant-label {subject_id} ' \
                 '--derivatives /opt/derivatives/precomputed -w /scratch --surface-recon-method mcribs --anat-only

What version of NiBabies are you using?

nibabies:23.1.0rc1

Relevant log output

231112-91:09:7,756 nipype.workflow IMPORTANT:
Running nibabies version 23.1.0rc1:
* BIDS dataset path: /data.
Participant list: [['1123',newborn']].*
*Run identifier: 20231112-010306 f834c77f-d7ea-43c8-89a2-87ec0a6ca8b3.
* Output spaces: MNIInfant.* Pre-run FreeSurfer's SUBJECTS DIR: /out/sourcedata/freesurfer.231112-91:09:7,872 cli WARNING:
-age-months' is deprecated and will be removed in a future release.please use a 'sessions.tsy' orparticipants,tsy' file to track participants age.Traceback (most recent call last):File "/opt/conda/envs/nibabies/bin/nibabies", line 8,in <module>
svs.exit(main())
File "/opt/conda/envs/nibabies/lib/python3.10/site-packages/nibabies/cli/run,py", line 63,in mainretval = build workflow(confia file)
File "/opt/conda/envs/nibabies/lib/python3.10/site-packages/nibabies/cli/workflow.py", line 80, in build workflow
retvall"workflow"1 = init nibabies wf(config,execution,unigue labels)File "/opt/conda/envs/nibabies/lib/python3.10/site-packages/nibabies/workflows/base.py", line 130, in initnibabies wf

Add any additional information or context about the problem here.

No response

@yanbin-niu yanbin-niu added the bug Something isn't working label Nov 12, 2023
@mgxd
Copy link
Collaborator

mgxd commented Nov 12, 2023

We have reworked --derivatives in #305 - you will likely have to update your derivatives space, and add the SpatialRefernce metadata.

Thank you for testing out the prerelease - looks like we will need to update the documentation prior to the actual release 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants