Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Updated rmg_runner.py and test_rmg_runner.py #86

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Updated rmg_runner.py and test_rmg_runner.py #86

wants to merge 3 commits into from
+109 −4

Commits on Dec 18, 2022

  1. Updated rmg_runner.py and test_rmg_runner.py 

    rmg_runner.py: File has been updated with assertion checks to ensure the user is correctly choosing memory if they specifiy
test_rmg_runner.py: Test has descriptions and also checks for assertion errors when the memory is incorrectly specified.
    @calvinp0
    calvinp0 committed Dec 18, 2022
    Configuration menu
    Copy the full SHA
    2acce40 View commit details
    Browse the repository at this point in the history

Commits on Dec 19, 2022

  1. Updated test_rmg_runner.py 

    test_rmg_runner.py: Updated the class TestWriteSubmitScript with a function that runs executes t3 as a 'local' job, and then after 'converging' the function will ensure that the job.sh and submit.sub files are written as expected.
    @calvinp0
    calvinp0 committed Dec 19, 2022
    Configuration menu
    Copy the full SHA
    a80cc69 View commit details
    Browse the repository at this point in the history

  2. Fixed a os.remove error 

    os.remove error that there was no job.sh to remove. Therefore, removed the function.
    @calvinp0
    calvinp0 committed Dec 19, 2022
    Configuration menu
    Copy the full SHA
    c3d43b4 View commit details
    Browse the repository at this point in the history