Skip to content

Updated ARC Zeus submit script#42

Open
alongd wants to merge 1 commit intomainfrom
arc_zeus_sb
Open

Updated ARC Zeus submit script#42
alongd wants to merge 1 commit intomainfrom
arc_zeus_sb

Conversation

@alongd
Copy link
Copy Markdown
Contributor

@alongd alongd commented Apr 27, 2026

corrected arc_code to arc_path, added n170

corrected arc_code to arc_path, added n170
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_code to $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
Copy link

Copilot AI Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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 ~.

Suggested change
python $arc_path/ARC.py input.yml
arc_path="${arc_path/#\~/$HOME}"
python "$arc_path/ARC.py" input.yml

Copilot uses AI. Check for mistakes.
@calvinp0
Copy link
Copy Markdown
Contributor

@alongd in #43 I made a change in the submit.py for molpro regarding the tmp folder to gtmp. could you include that in your PR instead here?

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 this pull request may close these issues.

3 participants