Adele Morrison

Results 41 comments of Adele Morrison

Maybe this could be good practice for the group doing PR review training to hone their skills on during the hackathon?

Oh yes, that's what a meant. A bunch of small PRs and then each person in the training could work on reviewing one each.

Regarding @anton-seaice's bug about finding the contour: - This now fails because `sc.get_paths()` now stacks all the contours instead of separating them into different paths. This is easy to fix...

Also with a comparison to observed summer / winter MLDs.

We want a config with interesting vertical coords (e.g. hybrid or rho not z*). Possibilities I can think of: * OM4. What vertical coord is this? Claire said it doesn't...

@claireyung @navidcy @edoddridge @rbeucher any takers?

True that this is very inefficient. But in this code, the mask is only computed once and then the very expensive part is calculating the cross-contour transport (often using daily...

I'm closing this issue. The code mentioned in this issue only takes 25s and only has to be done once and then the mask is usually saved to file. There...

Is this still relevant @aekiss ?

I think this notebook would be just as effective if it only dealt with 1 year of data. The advance, retreat and duration metrics can be calculated for a single...