McIntosh-Lab / tvb-ukbb

TVB-UKBB Pipeline: TheVirtualBrain implementation of the UK Biobank pipeline
Other
23 stars 12 forks source link

bad interpreter error in latest Cam-CAN commit #100

Closed shen4brains closed 3 years ago

shen4brains commented 3 years ago

This line throws a bad interpreter error in the latest Cam-CAN commit: https://github.com/McIntosh-Lab-RRI/tvb-ukbb/blob/8acb9220914cb00d1cb39c7881e7576006516a9d/bb_pipeline_tools/bb_pipeline.py#L1

Maybe because of directory mapping being relative to working directory:

-bash: /home/kshen/tvb-pipeline/tvb-ukbb/bb_pipeline_tools/bb_pipeline.py: ../../software/env/bin/python: bad interpreter: No such file or directory

It works for me when I revert to the previous commit

noahfl commented 3 years ago

Interesting - I don't encounter an error here regardless of where/how I run it. @shen4brains would you mind giving #!/../../software/env python a try instead when you get the chance? It may also just work without that line entirely

shen4brains commented 3 years ago

@noahfl Tried your first suggestion and got the same error. Then I removed the line entirely and the process seemed to get stuck immediately with no response in the terminal -- it looks like it might even be waiting for me to finish the command?

noahfl commented 3 years ago

Weird... I can't replicate this in my environment either. In that case @shen4brains we can just use the original line from the previous commit and we'll keep it that way for now. I think the reason I tried this was so all software references would be self-contained within the environment but that's super low priority if it works as-is

shen4brains commented 3 years ago

okay, changed it back to original line in commit 4fa1ff8