payu-org / payu

A workflow management tool for numerical models on the NCI computing systems
Apache License 2.0
19 stars 26 forks source link

make n_procs in cice_in.nml match ncpus in config.yaml #192

Open aekiss opened 5 years ago

aekiss commented 5 years ago

CICE will exit if n_procs in cice_in.nml doesn't match ncpus in config.yaml.

It would it be better for payu to modify n_procs in cice_in.nml to match ncpus in config.yaml.

marshallward commented 5 years ago

Somewhat related to this issue: https://github.com/payu-org/payu/issues/97

I agree this is very annoying and we should not have to suffer it.

I recall the debate was whether to just silently fix it or to warn the user and/or prevent submission. The challenge with the latter option is that we do far too much setup work inside the submission, when we probably ought to move that work prior to submission. I think this would actually solve a great many problems, since setup is where we see our errors.