Timothy Smith

I see what you're saying. My thinking is that by requiring the user to specify `doscaling=.True.` we are requiring them to come to terms with the fact that the weights...

Hmm, sorry An I don't mean to give you a hard time for such a relatively small change (in terms of line numbers) but I see the scaling operation a...

I'm sorry but unfortunately I wouldn't know how to do this at the moment. I have never looked into the profiles package.

Thanks @jm-c for the tag and apologies for delays here. I do remember originally thinking that the adcommon.h is not necessary, and then having the realization that any hand written...

Hey @hdrake! I think this would be a useful option to have, thanks! With regard to points 1 and 2, another option could be to read parameters from the STDOUT...

Tests currently fail due to the issue described in #312. They do pass when the pytest fixture scope is set to "function", as described in that issue.

Hi @ruth-moorman, does it work if you add the arguments `geometry="llc", nx=270`?

Hi @ruth-moorman, does it work to either not specify `iters`, or specify `iters=[iterations[0]]`? The type specification for the `iters` argument is a list, so this could be it. That's just...

Hi @ruth-moorman, too bad that wasn't the issue. I'm not really sure what's going on. I cannot reproduce the error using the curvilinear_leman dataset in xmitgcm's test suite. If there's...

Hey @cspencerjones! Does it work to specify your own inherited llc model on the fly, e.g. like ```python from xmitgcm import llcreader class SurfaceLLC4320Model(llcreader.LLC4320Model): varnames = ["SSU", "SSV"] ``` If...