Ed Doddridge

The variables `initHfile`, `initUfile`, and `initVfile` all have a lower case "f", while `initEtaFile` uses an upper case "F". These variables should use a single standard, and the rest of...

As identified in #121, the length of character variables used to store the input filenames can be an issue if the names become quite long. A quick fix of increasing...

PR #89 required the removal of Valgrind from the test suite because it wouldn't work with mpi as implemented. We should put it back, and make sure it can play...

The simulated momentum is different from the expected momentum (n-layer physics)

1

As indicated in #99, the tests developed for #66 have identified a discrepancy between the expected momentum and the simulated momentum in the n-layer configuration that is approximately a factor...

Model crashes if `dumpFreq` < `dt`

2

I've just encountered an annoying scenario in which the model fails with a very unhelpful error message of "Program received signal SIGFPE: Floating-point exception - erroneous arithmetic operation." It occurs...

Output model data in a portable and friendly format

5

The output are currently dumped as unformatted Fortran native files. These come with no metadata, are not particularly friendly to deal with, and may not be portable across systems. Other...

Add a suite of low-dependency user-runnable self-tests

1

Having two of the tests fail because of a missing dependency (namely, valgrind) is suboptimal. We should definitely keep valgrind in the test suite, but perhaps we don't set it...

Xgcm operations

23

First pass at a notebook that pulls in all the metrics required to create an `xgcm` grid object. It's all pretty rough and ready at the moment. The MOM6 example...