Taimoor Sohail
Taimoor Sohail
> so this is not an issue if you use conda envs 24.01 or earlier? Correct
I thought I pushed from taimoorsohail:main to Cosima-recipes:main? Either way, no, I wasn't trying to push in this branch. I was reviewing this PR request but there are significant bugs...
> [@taimoorsohail](https://github.com/taimoorsohail) what do you mean by "I currently define"? Are you referring to how the averaging is done in the [along-isobath averaging recipe](https://cosima-recipes.readthedocs.io/en/latest/Recipes/Along_Isobath_Average.html) at the moment? Yes that is...
> Variables like `temp` have a thickness defined by `dzt`, which at the bottom is aligned with `ht`, not `st_edges_ocean`. They're just output on the `st_ocean` grid for ease of...
Thanks, I'll work on this and get back to you.
It might also be worth plotting the surface integrated heat fluxes (i.e. `sum(Q*Az)`) over time to compare with the ECCO heat fluxes, if we are concerned it's a global bias...
Tests are still failing - having a look now
I haven't been able to figure out why these tests are failing - other than that they don't seem to be related to the NaN checker changes as far as...
Is this ready to merge once tests pass? now that #4520 is merged?
Thanks!