Adele Morrison

Results 111 comments of Adele Morrison

I think what Hannah was getting at is we need to provide an example of how to postprocess the zarr files to reduce the file numbers before they are transferred...

@hrsdawson do you know if progress on this was made at the Hackathon? Doesn't seem like it would take too long to add this warning and link, then we can...

Agreed Navid. We already have an issue for this: https://github.com/COSIMA/cosima-recipes/issues/435.

I think we should use the correct cell thickness (i.e. not use st_ocean_edges for binning), because otherwise locations could get put in the wrong isobath bins, with an error of...

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 saving.

> So, to clarify, the st_ocean dimension in temp is incorrect, and to be completely accurate, temp actually maps onto a variable vertical grid defined as the vertical cumulative sum...

I think it would be ok to ignore the variation in vertical position of cells above the bottom, because that doesn't vary very much. But binning using `ht`/`hu` would be...

Yeah I guess I was thinking under that section (or elsewhere), we could explicitly mention that this should (always?) be used for sea ice variables.

I think this still needs doing. Maybe the best place to add explanatory text that `compat="override"` is needed is in Sea_Ice_Coordinates.ipynb ?

@Thomas-Moore-Creative I found `%%timeit` is not useful if you are timing opening data from Intake or the cookbook, because the first time is very slow and repeat tests are very...