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

scaffold does not support over ride #170

Closed
GMerakis opened this issue Jun 12, 2022 · 2 comments
Closed

scaffold does not support over ride #170

GMerakis opened this issue Jun 12, 2022 · 2 comments

Comments

@GMerakis
Copy link
Contributor

right now it's only callable from command line.
This does not play nicely with scaling and calling from scripts.

GMerakis added a commit to GMerakis/Dcm2Bids that referenced this issue Jun 12, 2022
@DasDominus
Copy link

I needed to run these conversions in large scale and in my scripts.
Found that directly changing source & call from code is easier than docker (where I need to spin up subproccess)

I created a fork to customize for my non-docker needs.
I see no need for it right now, but once done I'll write a doc for non-docker walk-through also
https://github.com/DasDominus/NonDocker_Dcm2Bids

arnaudbore added a commit that referenced this issue Jun 17, 2022
Thank you @GMerakis for this PR. I think this will be the last PR and release before jumping to dcm2bids 3.0.0 with a cleaner and more flexible API.
Support output_dir override #170
@arnaudbore
Copy link
Contributor

@DasDominus, I encourage you to submit a PR if you have ideas to improve dcm2bids.
That will help the dcm2bids community.

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

No branches or pull requests

3 participants