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

change default pv colormap to grayscale #74

Closed

Conversation

keflavich
Copy link
Contributor

for purpose of better talk figure

also fix a deprecation warning in the ds9 script
@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 81.767% when pulling 03aa87a on keflavich:default_cmap_gui into 194f049 on radio-astro-tools:master.

@astrofrog
Copy link
Member

Aw, why not Viridis? (jk)

@keflavich
Copy link
Contributor Author

b/c c&p plt.cm.gray was easier. viridis is fine - want it instead?
dropdown would be sweet but that's too hard

On Wed, Mar 23, 2016 at 11:23 AM, Thomas Robitaille <
notifications@github.com> wrote:

Aw, why not Viridis?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#74 (comment)

Adam

@astrofrog
Copy link
Member

No, Viridis would require Matplotlib >1.5, gray is fine :)

@astrofrog
Copy link
Member

Closing in favor of #95

@astrofrog astrofrog closed this Sep 8, 2019
@keflavich keflavich deleted the default_cmap_gui branch November 22, 2021 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants