-
Notifications
You must be signed in to change notification settings - Fork 132
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
Docker documentation #1363
Comments
Full output/error.
Leaving off the |
Thanks! Yes, we do need to update the documentation. I think the right way to do this is: (i) keep the recommendation of :ro, but (ii) extend it to say that, if you plan to use datalad and have MRIQC datalad-get data, then of course you need to download the data somewhere and that somewhere cannot be protected with :ro. Another thing that you are not reporting here and IS NOT in the docs, is that you want to run docker with a given user (and potentially group). That can be done with The nipreps' docs are here https://www.nipreps.org/apps/docker/, probably worth updating there and reducing anything there is in MRIQC's docs to a link to this page. |
Closing as this has been addressed with #1367 and nipreps/nipreps.github.io#47. |
What would you like changed/added and why?
With the update of MRIQC enabled to incorporate datalad and be able to pull data down from OpenNeuro using datalad, the Docker instructions with the
:ro
(readonly) tag for the BIDS directory volume fail. I don't have the exact error message currently (macOS update woes), but it was something along the lines of trying to download on a read only filesystem.I can update the documentation, as long as there's no potential problems with dropping the
:ro
in the volume binding for the BIDS directory.Do you have any interest in helping improve the documentation?
Yes
Do you have any suggestions for the new documents?
@oesteban tagging for your thoughts.
No response
The text was updated successfully, but these errors were encountered: