-
Notifications
You must be signed in to change notification settings - Fork 31
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.
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
Nohup #75
Nohup #75
Conversation
…up mode with this niceness. Also convert nohup and nice from str to bool in runclaw since Makefile passes in string False rather than boolean.
@@ -25,6 +25,17 @@ def runclaw(xclawcmd=None, outdir=None, overwrite=True, restart=False, | |||
|
|||
If rundir is None, all *.data is copied from current directory, if a path | |||
is given, data files are copied from there instead. | |||
|
|||
If print_git_status is True, print a summary of the git status of all |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
useful!
to include other PRs that have been merged. * master: Change .f to .f90 in conversion Makefiles fixed conversion/Makefile_amrclaw_2d to point to f90 rather than .f files allow environment passed in to make_all so they can be set in run_tests.py
This PR was updated in April so that it doesn't always use nice with nohup, and some other improvements. It's been used fairly extensively since then on a probabilistic tsunami study, so I suggest merging in unless there are more suggestions. |
This looks good to me and will be a nice additional feature. |
Don't let me get in the way of merging this PR. I just want to point out that the default behavior from the Makefile is to now call the program with |
…onvert any non-integer to None (including string 'None')
Good catch @ahmadia. That wasn't the intention and I fixed it so None is the default. |
Does that do what you want? I can't figure out from looking at the code so I'm trying to run a test case. |
@rjleveque - I don't think does what you want. Re-reading the code, you're passing "None" is as a string, which won't be automatically converted into the None type. I know the options-parsing in Python is not great, but this code would really benefit from refactoring either into the optparse (portable) or argparse ( |
@ahmadia, yes I agree it should be cleaned up at some point, but I thought it worked now with the change to runclaw.py that's also in this commit, which resets the string to a proper None. |
Sorry, I missed that. The logic looks correct now, and it sounds like you're using this in production. I don't really maintain |
For running in batch mode on remote machines it is useful to be able to run the code in nohup mode (keep running if connection is broken) and to nice the job.