Conversation
corrected arc_code to arc_path, added n170
There was a problem hiding this comment.
Pull request overview
Updates the Zeus ARC PBS submit script to target the intended queue/host and to use the correct ARC installation path variable from the Zeus environment setup.
Changes:
- Switch PBS queue to
alon_q. - Constrain the job to run on host
n170. - Update ARC invocation from
$arc_codeto$arc_path.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| python $arc_code/ARC.py input.yml | ||
|
|
||
| python $arc_path/ARC.py input.yml |
There was a problem hiding this comment.
submit.sh calls python $arc_path/ARC.py ..., but in Servers/Zeus/.bashrc this repo exports arc_path as '~/Code/ARC/' (tilde inside quotes). Bash does not perform tilde expansion on values coming from variables, so this can resolve to a non-existent path at runtime. Prefer exporting arc_path as an absolute path (e.g., using $HOME), or set/normalize arc_path in this script before invoking Python so the job doesn't fail due to an unexpanded ~.
| python $arc_path/ARC.py input.yml | |
| arc_path="${arc_path/#\~/$HOME}" | |
| python "$arc_path/ARC.py" input.yml |
corrected arc_code to arc_path, added n170