Skip to content

DOC: Update link to IBM's colormap selection paper#11799

Merged
NelleV merged 2 commits into
matplotlib:masterfrom
saketkc:patch-3
Aug 18, 2018
Merged

DOC: Update link to IBM's colormap selection paper#11799
NelleV merged 2 commits into
matplotlib:masterfrom
saketkc:patch-3

Conversation

@saketkc

@saketkc saketkc commented Jul 30, 2018

Copy link
Copy Markdown
Contributor

PR Summary

Small doc change that fixes broken link to IBM's color selection paper.

PR Checklist

  • Has Pytest style unit tests
  • Code is PEP 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

@timhoffm

Copy link
Copy Markdown
Member

Thanks for the contribution!

Even though http://www.research.ibm.com/people/l/lloydt/color/color.HTM looks like a specialized address, it seems to just bring me to the general reseach entry page of IBM. Can you please recheck this?

@saketkc

saketkc commented Jul 30, 2018

Copy link
Copy Markdown
Contributor Author

Hi @timhoffm, that's correct. That seems to be a default page for a 404. For example:
https://www.research.ibm.com/whatever.html

@matthew-brett

Copy link
Copy Markdown
Contributor

If you search the IBM site for "select colormap" you get the page as the top hit, with the address you've given, so I think the link is correct, but IBM's site isn't working correctly.

@timhoffm

timhoffm commented Aug 2, 2018

Copy link
Copy Markdown
Member

The article seems to be gone from the IBM site. http://www.citeulike.org/user/akavipat/article/5983364 references the article via the similar address http://researchweb.watson.ibm.com/people/l/lloydt/color/color.HTM, which also just gets redirected to the start page.

There's a copy on github: https://github.com/ResearchComputing/USGS_2015_06_23-25/blob/master/25_June/ColorTheory_References/Why%20Should%20Engineers%20and%20Scientists%20Be%20Worried%20About%20Color.pdf, but I'm not sure we want to link to that.

@saketkc

saketkc commented Aug 2, 2018

Copy link
Copy Markdown
Contributor Author

Will the archive link not be okay?

The paper seems to be here (b & w): http://citeseerx.ist.psu.edu/viewdoc/download?doi=10.1.1.452.1807&rep=rep1&type=pdf

@jklymak

jklymak commented Aug 2, 2018

Copy link
Copy Markdown
Member

Please reference by DOI if available, then we aren't at the mercy of changing websites. If there is a free version that can be linked as well thats fine: http://ultra.sdk.free.fr/docs/Image/Colors/A%20Rule-based%20Tool%20for%20Assisting%20Colormap%20Selection.htm

http://dx.doi.org/10.1109/VISUAL.1995.480803

@jklymak jklymak added this to the v2.2-doc milestone Aug 10, 2018
@NelleV NelleV self-requested a review August 18, 2018 17:36
@NelleV NelleV merged commit 1c50036 into matplotlib:master Aug 18, 2018
@NelleV

NelleV commented Aug 18, 2018

Copy link
Copy Markdown
Member

Thanks @saketkc !

QuLogic pushed a commit to QuLogic/matplotlib that referenced this pull request Aug 19, 2018
DOC: Update link to IBM's colormap selection paper
tacaswell pushed a commit to tacaswell/matplotlib that referenced this pull request Nov 9, 2018
DOC: Update link to IBM's colormap selection paper
@tacaswell tacaswell mentioned this pull request Nov 9, 2018
tacaswell pushed a commit to tacaswell/matplotlib that referenced this pull request Nov 10, 2018
DOC: Update link to IBM's colormap selection paper
tacaswell added a commit that referenced this pull request Nov 10, 2018
DOC: Backport colormap documentation

Includes  #11905, #11799, #11881, and #12084
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants