T3 rmg zeus submit script#7
Merged
Merged
Conversation
898d15a to
c9c1787
Compare
c9c1787 to
9d34493
Compare
There was a problem hiding this comment.
Pull request overview
This PR adds Zeus-specific local T3 configuration files (submit script + settings) intended to be placed under ~/.t3/ for running RMG via PBS on the Zeus cluster, and it updates the Zeus .bashrc to avoid user-specific manual edits.
Changes:
- Add
Servers/Zeus/.t3/t3_submit.pydefining a PBS submit template for T3’srmgjobs. - Add
Servers/Zeus/.t3/t3_settings.pyoverriding T3 runtime/server settings for Zeus. - Update
Servers/Zeus/.bashrcto use$USERfor thestalias instead of a placeholder.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| Servers/Zeus/.t3/t3_submit.py | Adds a Zeus PBS submit-script template used by T3 when submitting RMG jobs. |
| Servers/Zeus/.t3/t3_settings.py | Adds Zeus-specific T3 settings overrides (execution mode, server resources, PBS commands). |
| Servers/Zeus/.bashrc | Removes the manual username placeholder by switching st to qstat -u $USER. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| touch initial_time | ||
|
|
||
| python-jl $rmgpy_path/rmg.py -n {cpus} input.py{max_iterations} |
Comment on lines
+20
to
+24
| 'local': { # Each Zeus node has 80 cores and 378 GB RAM | ||
| 'cluster_soft': 'PBS', | ||
| 'cpus': 16, | ||
| 'max mem': 40, # GB | ||
| }, |
Adds Servers/Zeus/.t3/ — the T3 counterpart of the existing .arc config — so T3 runs on Zeus without hand-edited settings: - t3_settings.py: PBS server + Zeus /opt/pbs/bin command paths (matching the ARC Zeus config); RMG submitted to the queue, ARC run in-process. Keys/values match what T3 actually reads (execution_type incore|local, servers 'max mem', check_status/submit/submit_filenames). - t3_submit.py: PBS RMG submit template using $rmgpy_path, pinned to node n170 on alon_q like Servers/Zeus/ARC/submit.sh; filled by T3 via name/cpus/memory/workdir/max_iterations (max_iterations placed as its own token so spacing is safe). Files are named t3_settings.py / t3_submit.py, the names T3 loads from ~/.t3 (t3.imports); the earlier draft's settings.py would never load. Drops python-jl (no longer used): the RMG submit template, the RMG/submit.sh script, and the rmg/arkane .bashrc aliases now call plain python. RMG/submit.sh also now uses $rmgpy_path (the exported var) instead of the undefined $rmg_code. .bashrc: `st` alias uses `$USER` instead of a hand-edited <user>. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
9d34493 to
199bc1c
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds the t3_submit and the settings file for .t3 folder. Also updates .bashrc so that no manual editing is required.