Skip to content

Update sapelo_programs.py - #49

Open
cgbriggs99 wants to merge 11 commits into
masterfrom
cgbriggs99-patch-2
Open

Update sapelo_programs.py#49
cgbriggs99 wants to merge 11 commits into
masterfrom
cgbriggs99-patch-2

Conversation

@cgbriggs99

Copy link
Copy Markdown
Contributor

Update software versions and fix an issue with the ORCA format string where {} is treated like it needs an argument.

Update software versions and fix an issue with the ORCA format string where `{}` is treated like it needs an argument.
Comment thread submitscripts/slurm/sapelo_programs.py Outdated
Comment thread submitscripts/slurm/sapelo_programs.py
Comment thread submitscripts/slurm/sapelo_programs.py Outdated
@AlexHeide
AlexHeide dismissed their stale review May 12, 2026 19:06

Changes made. Resting now.

@MitchLahm

Copy link
Copy Markdown
Collaborator

The changes generally look good to me. Could we change the the "molpro" option in progdict to be "molpro_21" so it's a bit more clear which version the more antiquated version is in comparison to "molpro_24?"

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