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

patching singularity containers #1338

Closed
jdkent opened this issue Oct 22, 2018 · 1 comment · Fixed by #1340
Closed

patching singularity containers #1338

jdkent opened this issue Oct 22, 2018 · 1 comment · Fixed by #1340

Comments

@jdkent
Copy link
Collaborator

jdkent commented Oct 22, 2018

In the current contributors.rst, it says to use the PYTHONPATH variable to patch singularity containers.

Patching containers can be achieved in Singularity by using the PYTHONPATH variable:

$ PYTHONPATH="$HOME/projects/fmriprep" singularity run fmriprep.img \
     /scratch/dataset /scratch/out participant -w /out/work/

This method did not work for me (but I didn't try hard to make it work). However binding did work, and it's synonymous with how we patch with docker containers.

e.g.

singularity run \
-B $HOME/projects/fmriprep/fmriprep:/usr/local/miniconda/lib/python3.6/site-packages/fmriprep \
fmriprep.img \
/scratch/dataset /scratch/out participant -w /out/work/

This would be my preferred way of patching singularity containers, but I want to get other opinions before opening up a pull request to change the docs.

@oesteban
Copy link
Member

+1 to update the guide and place your method 1st (although keeping the PYTHONPATH mechanism)

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