cosima-recipes icon indicating copy to clipboard operation
cosima-recipes copied to clipboard

Update `SeaIceSeasonality_DFA.ipynb` notebook

Open navidcy opened this issue 1 year ago • 1 comments

At the moment this notebook does not run. We need to update it and then add it to the list of automated tests in https://github.com/COSIMA/cosima-recipes/blob/main/test/test_notebooks.py

If we can make it work with MOM6 output it'd be even better!

navidcy avatar Sep 02 '23 20:09 navidcy

Also good idea is to deal with #267 while updating the notebook

navidcy avatar Sep 02 '23 20:09 navidcy

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

adele-morrison avatar Aug 30 '24 21:08 adele-morrison

It's currently breaking on this line: time = pd.date_range(str(year)+'-02-15', str(year)+'-02-16', freq = 'D', closed = 'left') with error TypeError: DatetimeArray._generate_range() got an unexpected keyword argument 'closed'

adele-morrison avatar Aug 30 '24 21:08 adele-morrison

It's also quite elaborate recipe!

I opened it to attempt to convert it and I was so intimidated that I closed it...

navidcy avatar Aug 30 '24 21:08 navidcy

Same :)

adele-morrison avatar Aug 30 '24 21:08 adele-morrison