Skip to content
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

fix #238 for @lee1043 based on @durack1 recommendation to not load it… #244

Merged
merged 8 commits into from Sep 2, 2017

Conversation

@doutriaux1
Copy link
Contributor

@doutriaux1 doutriaux1 commented Sep 1, 2017

… a start time but only on demand

@lee1043 does this looks like what you want simply call:

import vcs
vcs.utils.loadmatplotlibcolormaps()
doutriaux1 added 2 commits Sep 1, 2017
… a start time but only on demand
@lee1043
Copy link
Contributor

@lee1043 lee1043 commented Sep 1, 2017

@doutriaux1 Good to have it. I am sure it makes having matplotlib cmaps much easier than converting individuals by users. Thanks!

@durack1
Copy link
Member

@durack1 durack1 commented Sep 1, 2017

@doutriaux1 excellent!

Can we add in all the auxiliary palettes, so we have the entire kitchen sink (Ken's, Karl's (#238 (comment)), mine (#238 (comment)) etc)

doutriaux1 added 6 commits Sep 1, 2017
only 1 test at a time longer but easier to see output
@doutriaux1 doutriaux1 merged commit d7ac71f into master Sep 2, 2017
2 checks passed
2 checks passed
ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@doutriaux1 doutriaux1 deleted the mpl_cmaps branch Sep 2, 2017
@doutriaux1 doutriaux1 added this to the 2.12 milestone Sep 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants