Skip to content

Skip mris_expand on pre-existing graymid/midthickness files #477

Closed
@effigies

Description

@effigies

With #476, I'm remembering that mris_expand gets called every time the scratch directory gets wiped out, even though the output (midthickness files) exists. This is a roughly ~12min step. I'm not sure what the best way to skip this would be.

One option would be: Add a graymid/midthickness output to nipype.interfaces.io.FreeSurferSource. Then subclass MRIsExpand to take that field as an input. If it exists, change the command to cp $GRAYMID $HEMI.midthickness.

In general I don't love that we're writing into a subject directory, but for reasons explored in #398 (comment) it's a least-bad option. But especially if somebody provides a pre-computed midthickness file, I think it's better not to replace it, and the above proposal overwrites it, but with a copy of itself, so it's pretty tame.

Any other thoughts?

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions