Skip to content

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

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

Merged
merged 2 commits into from
Aug 18, 2018
Merged

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

merged 2 commits into from
Aug 18, 2018

Conversation

saketkc
Copy link
Contributor

@saketkc saketkc commented Jul 30, 2018

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
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
Copy link
Contributor Author

saketkc commented Jul 30, 2018

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
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
Copy link
Member

timhoffm commented Aug 2, 2018

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
Copy link
Contributor Author

saketkc commented Aug 2, 2018

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
Copy link
Member

jklymak commented Aug 2, 2018

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
Copy link
Member

NelleV commented Aug 18, 2018

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