Timothy Smith
Timothy Smith
Nice @mjlosch! All of the deprecated ctrl-smooth code is gone - it looks very clean. I also think that `ctrl_getrec` should be removed, since it is not called by any...
@mjlosch regarding the seaice cost files, I think that this should be left in because it's not abundantly clear to me that the same functionality is implemented somewhere else in...
@mjlosch hmm that is a tricky one, I didn't realize this was turned on in the verification_other experiment. There is no way to apply the smoother twice by providing options...
I totally agree with @jm-c > If we need to update results it means that we are not just removing deprecated code It seems best to tackle this in a...
Thanks @mjlosch, that is good to hear - the Jenga tower is back up once again!
I agree, I think this would be helpful.
Thanks for this PR @owang01, I think it will be useful and clean. I do agree with @mjlosch though, I think the implementation would be cleaner by explicitly setting the...
Thanks @owang01, I think this looks good. It is much easier to see what is going on just while looking through the code. I'll make some fairly detailed comments but...
I have to say I don't really understand why the [tutorial_plume_on_slope check is failing](https://travis-ci.org/github/MITgcm/MITgcm/jobs/717641038). I ran the testreport on the cluster at our institute and without turning on mpi it...
Hey @mjlosch, thanks so much for the feedback and helpful debugging! Yes I'd be interested to see your fix. I'm re-wrapping my head around all these buffers, segs, and pass...