PennLINC / babs

BIDS App Bootstrap (BABS)
https://pennlinc-babs.readthedocs.io
MIT License
5 stars 5 forks source link

[FIX] directives for interpreting shell #94

Closed zhao-cy closed 1 year ago

zhao-cy commented 1 year ago

Currently in SGE version, would have:

#!/bin/bash
#$ -S /bin/bash

Where 1st line was added automatically by BABS, and 2nd line is from interpreting_shell. This is redundant. Also, not sure what's the #SBATCH version of this for Slurm clusters. Therefore, probably better just:

  1. not to automatically generate #!/bin/bash
  2. use interpreting_shell to generate #!/bin/bash or #!/bin/bash -l (for Slurm) - based on what users give in interpreting_shell
zhao-cy commented 1 year ago

Question: does container_zip.sh also needs to use this directives? Or does not matter, only participant_job.sh like this would be enough?

zhao-cy commented 1 year ago

TODO: