post_op about dcm2bids

[arnaudbore]

Perfect, everything you see in this PR will be in the next release end of this week I think.

[arnaudbore]

Thank you @raniaezzo for reporting this issue. I'll try to replicate it asap and fix it if needed.

[arnaudbore]

@raniaezzo can you try with the PR #260 ? I think this should fix your issue.

[raniaezzo]

@arnaudbore Yes, thanks. It fixed the original issue which was about getting pydeface to run. However, after it completes, the output is not actually defaced when doing it through dcm2bids function.

When I run dcm2bids, the output includes:

INFO | Running: pydeface --outfile /Users/rje257/Desktop/BIDS_testing/sub-0325/ses-01/anat/sub-0325_ses-01_run-01_T1w.nii.gz /Users/rje257/Desktop/BIDS_testing/tmp_dcm2bids/sub-0325_ses-01/032_Subject_0325_anat-T1w_20221003112735.nii.gz --verbose --debug --force
. . .
(it pauses and takes a while during the defacing step)
. . .
INFO | --- dcm2bids end ---

But the output /Users/rje257/Desktop/BIDS_testing/sub-0325/ses-01/anat/sub-0325_ses-01_run-01_T1w.nii.gz is NOT defaced.

I copied the output file back into the tmp directory and re-ran the pydeface command directly (not through dcm2bids), and the defacing works.

[arnaudbore]

I fixed the issue I think you are referring. Can you check you test the PR with the latest commit.

commit 2dc01db96bcbd505ef6559e6ebffb877990582f0 (HEAD -> fix_post_op, origin/fix_post_op)
Author: Arnaud Bore <[email protected]>
Date:   Mon Sep 11 17:14:15 2023 -0400

    change cp to shutil copy

[raniaezzo]

Yes, that worked. Thanks!