-
Notifications
You must be signed in to change notification settings - Fork 70
INTAKE_Fig12-ZonelTempSalt #524
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
@@ -2,6 +2,7 @@ | |||
"cells": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we could probably speed this up a good bit by providing some sensible chunking.
Reply via ReviewNB
@@ -2,6 +2,7 @@ | |||
"cells": [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line #3. ds = xr.open_mfdataset('/g/data/ik11/observations/woa13/01/woa13_ts_??_mom01.nc',chunks=woa_chunks)
We can get rid of this open_mfdataset
call once we have the esmvaltool-obs dataset in the catalog right?
Reply via ReviewNB
I think we can clean this up a bit - especially related to chunking, etc. Jumping in now |
@rbeucher It seems this experiment is no longer in the catalog. Presumably the paper we are reproducing figures from is this one?. Would @aekiss or @AndyHoggANU be the right people to ask about whether we would expect this experiment to still be in the catalog & if not what it wants replacing with? ![]() EDIT: Anyone looking at this far in the future will notice that I accidentally put the string |
Yeah, that is the right paper but the experiments are quite old. So, we don't expect to replicate the actual figure, just the method to find it, and importantly that it includes 3 different experiments - one at each resolution. You'll see we picked three relevant experiments for the other recipes in this directory (e.g. Fig. 3):
I would just use them, and choose the same time period (1993-2017) for the 4th cycle .... |
FYI details on current control experiments are here https://forum.access-hive.org.au/t/access-om2-control-experiments/258 |
Hi @charles-turner-1
I found out that @max-anu had converted that one.
Can you review please?