Intake conversion Model_Agnostic_Analysis #378
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:42Z We should add the conda environment that we need to use to run it. |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:43Z I prefer to have a chunk that only loads libraries and any use of functions (line 5) I like to have in a separate chunk. While trying to run the notebook in Gadi, I got an error in line 5 because I was not a member of project xp65. We should probably add this information to the introduction to make sure users can run the notebook. navidcy commented on 2024-07-29T05:14:16Z Agree |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:43Z When this address "tcp://10.6.77.34:8786" is used in the navidcy commented on 2024-07-29T05:14:27Z I doubt |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:44Z This description does not appear to match what the chunk below is doing. Are you printing the variables available in the catalog? navidcy commented on 2024-07-29T05:14:34Z I rephrased |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:45Z This chunk does not run because cat_subset is not defined until the next chunk. I would replace with: experiment = '025deg_jra55_iaf_omip2_cycle6' cat_subset = catalog[experiment] sorted(set().union(*cat_subset.df['variable'])) Then the chunk below will look like the following: variable = 'sst' var_search = cat_subset.search(variable=variable, frequency='1mon') darray = var_search.to_dask() darray = darray[variable] SST = darray navidcy commented on 2024-07-29T05:14:59Z I doubt the |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:45Z We could probably replace the lines below: darray = darray[variable] SST = darray with the following: SST = darray[variable] navidcy commented on 2024-07-29T05:15:09Z of course! |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:46Z Since the units have not been updated, I would also add a note about the units in the colourbar are not correct. |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:47Z I would update the units after the conversion, so that is before creating the plot.
Also, I do not think this chunk is necessary because the units were already defined in the original dataset. |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:47Z I would update the name of the function from navidcy commented on 2024-07-29T05:18:43Z I disagree... the function will plot a global map if you provide with a global field, no? |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:48Z This experiment ( From this point onwards, the notebook does not run because either the experiments or variables used are not available in the catalogue. navidcy commented on 2024-07-29T05:18:59Z Sure... |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:49Z This experiment ( 025deg_jra55_iaf_omip2_cycle1 025deg_era5_iaf 025deg_jra55_iaf_omip2_cycle3 025deg_jra55_ryf9091_gadi 025deg_jra55_iaf_omip2_cycle2 025deg_jra55_iaf_omip2_cycle6 025deg_jra55_ryf_era5comparison 025deg_era5_ryf 025deg_jra55_iaf_era5comparison 025deg_jra55_iaf_omip2_cycle5 025deg_jra55_iaf_omip2_cycle4
Note that |
|
View / edit / reply to this conversation on ReviewNB lidefi87 commented on 2024-07-01T05:30:49Z Line #4. ice_air_temp = ice_air_temp.assign_coords({'TLON': ice_grid.tlon.pint.to('degrees_E'), 'TLAT': ice_grid.tlat.pint.to('degrees_N')})
This does not work because it is calling |
|
Agree View entire conversation on ReviewNB |
|
I doubt View entire conversation on ReviewNB |
|
I rephrased View entire conversation on ReviewNB |
|
I doubt the View entire conversation on ReviewNB |
|
of course! View entire conversation on ReviewNB |
|
I disagree... the function will plot a global map if you provide with a global field, no? View entire conversation on ReviewNB |
|
Sure... View entire conversation on ReviewNB |
|
Thanks @lidefi87 for your comments. I took them into consideration and pushed... but unfortunately we can't continue with this PR until MOM6 output is part of intake... |
|
No worries. I am happy that some of the comments were useful at least |
|
@dougiesquire, @anton-seaice do we have MOM6 output in the intake catalog or not yet? |
Not yet. Sounds like its close - should be this month I would guess. |
Following the discussion in issue #313, we propose converting the recipes to use Intake, given that the Cookbook is no longer supported and the ACCESS-NRI Intake catalog is now available.
A few months ago, @max-anu began working on this transition. This pull request contains the changes @max-anu made to the notebook specified in the title.