Skip to content

added optional argument 'subjects_dir' to project_volume_data #304

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mmbannert
Copy link

Dear PySurfers,

I have improved the "project_volume_data" function in surfer.io. It now accepts an additional optional argument "subjects_dir" of type str (default: None), which overrides the SUBJECTS_DIR environment variable in the call to Freesurfer's mri_vol2surf function by specifying its "--sd" argument.

I find this useful especially when, for instance, the Freesurfer code and the neuroimaging data live on different (remote) servers.

I have updated the documentation of that function by pasting the explanation of the same argument from PySurfer's "Brain" class. So it should be pretty clear what it does.

I hope you will find this useful!

Cheers,
Michael

Copy link
Contributor

@agramfort agramfort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but our CIs are in very bad shape...

@mwaskom @larsoner I think we could merge as is....

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants