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

added optional argument 'subjects_dir' to project_volume_data #304

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....

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

Successfully merging this pull request may close these issues.

3 participants