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

Add command line option for deriv_root #772

Closed
vferat opened this issue Jul 25, 2023 · 2 comments · Fixed by #773
Closed

Add command line option for deriv_root #772

vferat opened this issue Jul 25, 2023 · 2 comments · Fixed by #773

Comments

@vferat
Copy link
Contributor

vferat commented Jul 25, 2023

As I develop a Docker App using mne-bids-pipeline, I want a straightforward way to overwrite the bids_root and deriv_root values defined in the configuration file from the command line. This enhancement aims to simplify Docker volume mapping.

Currently, the option to override bids_root is already available (source), but there is no similar option for deriv_root.

I suggest adding a command-line option to the mne-bids-pipeline tool to allow users to specify the deriv_root directory. This behavior is similar to other BidsApps such as FMRIprep and MRIQC, where command line arguments overwrite configuration files.

If you are interested, I would be happy to contribute by taking care of the pull request.

@welcome
Copy link

welcome bot commented Jul 25, 2023

Hello! 👋 Thanks for opening your first issue here! ❤️ We will try to get back to you soon. 🚴🏽‍♂️

@larsoner
Copy link
Member

Makes sense to me, PR welcome!

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 a pull request may close this issue.

2 participants