Remove deprecated cosima_cookbook references from ACCESS-NRI catalog tutorial

[navidcy]

It's time to let go of the cosima_cookbook!

[review-notebook-app]

Check out this pull request on 

See visual diffs & provide feedback on Jupyter Notebooks.

---

Powered by ReviewNB

[navidcy]

The last cell fails

df = get_detailed_variable_info(catalog, experiment, 'tau_x')

But if all else is good, I suggest we merge and we fix that tomorrow?

[charles-turner-1]

Just coming into Melbourne now so I can't check, but the last cell looks like a relatively straightforward fix. I'll take a proper look later/first thing tomorrow.

[navidcy]

@charles-turner-1 whenever you find sometime have a look and if you fix the last cell feel free to approve and merge!

[charles-turner-1]

This should no longer be true and work out of the box.

Reply via ReviewNB

[charles-turner-1]

Line #3.        path=".*output000.*"

This line can be deleted (I've checked & it's not doing anything)

Reply via ReviewNB

[charles-turner-1]

if columns_with_iterables:
        df = df.explode(columns_with_iterables, ignore_index=True)
    df.index = df[variable_column_name]
Line #29.        df.index = df[variable_column_name]

I think that this will separately explode on each column, which is probably not what we want. It looks like @adele-morrison wrote this - are you able to confirm for me?

Reply via ReviewNB

[charles-turner-1]

Turns out I wrote this - @navidcy you happy for me push the changes to this branch?

[charles-turner-1]

def get_detailed_variable_info(intake_catalog, experiment_name : str, variable : str | None = None) -> "pd.Dataframe":
    """
    Get detailed information about all the variables available in an experiment contained within the catalog.

    If a specific variable is passed, then the returned dataframe will be filtered to include only information
    about that variable

    Returns a pandas dataframe, reorganised to use the variable as the index.

    Parameters:
    -----------
    intake_catalog: 
        The variable holding the intake catalog. If you have opened the catalog using
        `cat = intake.cat.access_nri`, then `intake_catalog=cat`, etc.
    experiment_name: str
        The name of the experiment you are interested in. Eg. `experiment = "01deg_jra55v13_ryf9091"`
    variable: str | None
        If you want detailed information about just a single variable, then pass it here. For 
        example, if you only want information about potential temperature, pass `variable='pot_temp'`
    """

    expt_ds = intake_catalog[experiment_name]
    columns_with_iterables = list(expt_ds.esmcat.columns_with_iterables)
    variable_column_name = expt_ds.esmcat.aggregation_control.variable_column_name
    
+    for col in columns_with_iterables:
+        expt_ds.df[col] = expt_ds.df[col].astype("object").map(lambda x: list(map(str, x)))

    df = expt_ds.df.copy()
    if columns_with_iterables:
        df = df.explode(columns_with_iterables, ignore_index=True)
        
    df.index = df[variable_column_name]
    df.drop(columns=variable_column_name, inplace=True)
    if variable is not None:
        df = df.loc[variable]

    return df

fixes the failing cell, but I'm not sure it gives the intended results.