search

payu-org / payu

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

Read MOM6 parameter filenames from experiment in cesm_cmeps driver #371

Closed dougiesquire closed 8 months ago

dougiesquire commented 9 months ago

The same issue described in #325 is also in the cesm_cmeps driver:

https://github.com/payu-org/payu/blob/cfd694aa4744b1c7578e18a8993650df9bd88bfe/payu/models/cesm_cmeps.py#L26-L32

In addition, field_table (and possibly data_table??) needs to be added to the "optional_config_files" key.

aidanheerdegen commented 8 months ago

Seem to be talking about mixin classes a lot lately, but would this be a good case, move the logic @jo-basevi just created to a mixin and inherit that in both models? Is there sufficient overlap of functionality?

dougiesquire commented 8 months ago

Sure - if it's possible to do this as a mixin, that would be great