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

Perceptibly uniform luts #6398

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
Open

Conversation

Tom-TBT
Copy link

@Tom-TBT Tom-TBT commented Jul 19, 2024

What this PR does

I added perceptibly uniform LUTs. I started with the LUTs from the viridis family that I found in Fiji, but I extended it to other LUTs used in R: https://github.com/sjmgarnier/viridis

Testing this PR

  1. required setup
    An OMERO.server

  2. actions to perform
    Add the new LUTs to OMERO.server/lib/scripts/luts

  3. expected observations
    The LUTs become available for all tools (but there is no preview of the LUT intensity from the dropdown lists):

  • OMERO.iviewer
  • OMERO web image viewer + preview tab
  • OMERO.insight
  • OMERO.figure

Related reading

Link to cards, tickets, other PRs:

  1. background for understanding this PR
    The viridis LUT family have been created with perception uniformity as criterion. They are the standard of matplotlib for some time now and are also packaged with Fiji. I think it would be good for users to find them also by default in OMERO.
    The cividis is the colorblind adjusted version of viridis which also has been integrated in Fiji (https://forum.image.sc/t/new-lut-for-fiji-cividis/7467).
    Seaborn also offers two perceptually uniform LUT, mako and rocket, part of the "viridis" package for R.
    Finally, turbo is an alternative to jet that is also more colorblind-friendly and gradual.

  2. what this PR assists, fixes, or otherwise affects
    This will package new LUTs with OMERO.server. The LUTs will become available for other OMERO components (web, iviewer, figure, insight) but will not display preview until related PR are merged and deployed.

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.

1 participant