-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add stripes fct #56
Add stripes fct #56
Conversation
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.
Il faudrait un exemple dans le notebook!
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.
The function doesn't work with a DataArray. It gives back a blank figure with only the time axis and colorbar.
Otherwise, it looks good!
Conflicts: spirograph/matplotlib/__init__.py spirograph/matplotlib/plot.py
Made a few changes:
Tested more thoroughly, everything seems to be working fine. |
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.
cool! J'écrirais juste les ssp un peu plus gros.
Added climate stripes function, without some of the stylistic elements dividing the historical and future periods (tbd).
Minor bug fixes in other functions.