-
Notifications
You must be signed in to change notification settings - Fork 19
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
Updates in preparation for next release #92
Conversation
Additional commentary: I tried to make as few changes as possible in this PR, and I think this is a release candidate for |
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.
Looks great, thanks!
Thank you @dkirkby. I'll leave this open longer in case anyone wants to take a look: @segasai, @moustakas, @araichoor. |
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 can see that you added the previously failing example to the tests, so it looks good to me in terms of fixing #34
Thanks!
Thank you everyone, merging and tagging momentarily. |
Could
|
Done, but only for |
This PR:
cm.get_cmap
#88speclite_benchmark
command.