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

Allow passing batch arguments through case.submit command line options #922

Closed
rljacob opened this issue Dec 14, 2016 · 2 comments
Closed

Allow passing batch arguments through case.submit command line options #922

rljacob opened this issue Dec 14, 2016 · 2 comments
Assignees
@jgfouca
Labels
ty: enhancement

Comments

@rljacob
Copy link
Member

rljacob commented Dec 14, 2016

Suggested by @erichlf over on ACME.

If one wants to add arguments to the batch submit command, they have to edit env_batch.xml. For experimenting and trying something once, it would be easier to pass additional commands as an argument to case.submit like this:
case.submit --batchargs '--mail-user=user@server.com --mail-type=END'
@erichlf
Copy link

erichlf commented Dec 14, 2016

The original PR can be seen here.

@jedwards4b
Copy link
Contributor

This seems like a good idea.

@jgfouca jgfouca mentioned this issue Feb 8, 2017
Merged
@ghost ghost assigned jgfouca Feb 8, 2017
@ghost ghost added the in progress label Feb 8, 2017
rljacob pushed a commit that referenced this issue Feb 8, 2017
@douglasjacobsen
Merge branch 'douglasjacobsen/scripts/change-mpas-g-cases' (PR #922)
68fbf86 
This merge adds an interannual forcing G compset, named GMPAS_IAF. It
also changes GMPAS to be GMPAS_NYF to avoid confusion.

[BFB]

* douglasjacobsen/scripts/change-mpas-g-cases:
  Add a GMPAS_IAF compset, and change GMPAS to GMPAS_NYF
rljacob pushed a commit that referenced this issue Feb 8, 2017
@douglasjacobsen
Merge branch 'douglasjacobsen/scripts/change-mpas-g-cases' (PR #922)
d3b6ab5 
This merge adds an interannual forcing G compset, named GMPAS_IAF. It
also changes GMPAS to be GMPAS_NYF to avoid confusion.

[BFB]

* douglasjacobsen/scripts/change-mpas-g-cases:
  Add a GMPAS_IAF compset, and change GMPAS to GMPAS_NYF
@jgfouca jgfouca mentioned this issue Feb 13, 2017
Merged
jgfouca added a commit that referenced this issue Feb 14, 2017
@jgfouca
Merge branch 'jgfouca/allow_arbitrary_batch_args' (PR #1137)
534aade 
Test suite: code_checker, by-hand
Test baseline:
Test namelist changes:
Test status: bit for bit

Fixes #922

User interface changes?: -a option to case.submit

Code review: @jedwards4b

* jgfouca/allow_arbitrary_batch_args:
  Add support to case.submit for sending arbitrary arguments to the batch submission cmd
@ghost ghost removed the in progress label Feb 14, 2017
@erichlf erichlf mentioned this issue Feb 17, 2017
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jgfouca jgfouca

Labels
ty: enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@rljacob @erichlf @jedwards4b @jgfouca