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

position probability plot: improve color map #186

Open
magland opened this issue Feb 10, 2022 · 7 comments
Open

position probability plot: improve color map #186

magland opened this issue Feb 10, 2022 · 7 comments

Comments

@magland
Copy link
Owner

magland commented Feb 10, 2022

Waiting for a high entropy example.

@magland
Copy link
Owner Author

magland commented Sep 9, 2022

@edeno @lfrank do you have a color map for this? Or should we mark this as low priority?

@edeno
Copy link

edeno commented Sep 9, 2022

How hard would it be for the user to be able to specify the colormap?

@jsoules
Copy link
Collaborator

jsoules commented Sep 9, 2022

Depends on what you want. We could allow the user to specify color map at figure create time, but adding a UI element to change it dynamically would probably be more trouble than it's worth. Additionally, choosing a good color map seems like a lot to ask for every user.

Another possibility is to use one of the perceptually uniform color maps (like we're doing for the track animation view). There are currently three or four of those supported, and while the choice is hard-coded right now, exposing it via code wouldn't be too hard.

@edeno
Copy link

edeno commented Sep 9, 2022

Yeah, I didn't mean for it to be dynamically choosable. I was just thinking of the options matplotlib provides: https://matplotlib.org/stable/tutorials/colors/colormaps.html which indeed could just be the perceptually uniform ones.

@jsoules
Copy link
Collaborator

jsoules commented Sep 9, 2022

I think we currently have imported magma, plasma, inferno, and viridis from the R viridis color maps, if that works. Track animation is currently set to magma I believe (with some amplification--for that view I use (value in (0, 255))/125, so roughly doubled its actual value, since the values tend to be very low overall.)

@jsoules
Copy link
Collaborator

jsoules commented Sep 9, 2022

@edeno
Copy link

edeno commented Sep 9, 2022

I think that should be sufficient.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants