search

aosprey / mckpp-f90

Multi-Column configuration of the K Profile Parameterisation mixed-layer ocean model (F90 version)
GNU General Public License v3.0
2 stars 0 forks source link

What to do about CAM coupling #23

Closed aosprey closed 1 year ago

aosprey commented 3 years ago

Currently the code supports coupling to CAM, but this is done through intrusive #ifdef statements throughout the infrastructure code, that make things hard to follow. Ideally we would re-do this with less need for #ifdefs. For the netcdf read code, I just made different versions of the routines rather than have #ifdefs altering nearly every line of code.

It is not clear if we even need to keep the CAM coupling in this version. Certainly it probably doesn't work with the developments I have made so far.

aosprey commented 1 year ago

I think we should remove the CAM interface. Since we are doing such a major code refactoring, it would be better to add in later if we need it.