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

Heat map center value feature request #4789

Closed
leeherman99 opened this issue May 18, 2020 · 8 comments · Fixed by #4809
Closed

Heat map center value feature request #4789

leeherman99 opened this issue May 18, 2020 · 8 comments · Fixed by #4809
Assignees

Comments

@leeherman99
Copy link

Is your feature request related to a problem? Please describe.

When generating Heat Maps, I would like to be able to display the data relative to some central value. For instance, if 1 is a "no effect" value, I'd like to see values >1 light up as red, <1 as green, for instance.

Describe the solution you'd like

I'm using the latest Orange available in Anaconda, v3.25.

Scaling in the heat map widget seems to be a moving target. You used to be able to center the colors around 0. That was sometimes useful when you had data that was actually centered around 0. Now that's gone. It would be nice to be able to center around an arbitrary value. For instance, I have some data that's centered around 0, others that is centered around 1.

Describe alternatives you've considered
My only alternative now is to use the sliders that control the High and Low values manually. However, that has to be changed as the data changes.

Additional context
Add any other context or screenshots about the feature request here.

@Hrovatin
Copy link
Contributor

For 0-centering you can use the Diverging pallets which are centered at 0. A work around for visually centering the palette at a specific value might be the use of Feature Constructor. A new feature could be generated so that it has 0 at the desired center of the original feature. However, this would be impractical for large number of features and the legend would be shifted due to the processing of the original feature.

@leeherman99
Copy link
Author

Thanks for the suggestion, but I'm in the situation you mentioned - lots of features. Also, that workaround is not useful when you want to examine the original values in the heatmap.

@janezd janezd added the needs discussion Core developers need to discuss the issue label May 21, 2020
@janezd janezd self-assigned this May 22, 2020
@janezd
Copy link
Contributor

janezd commented May 22, 2020

@leeherman99, we discussed this at today's meeting. This is one of "would be nice to have" features. We'll see whether it is easy to implement and incorporate into user interface. If it is (which will be the case, I'm guessing), we'll add it. If it would overly complicate the code or the interface, we'll decide it is not nice enough to have. :)

@janezd janezd removed the needs discussion Core developers need to discuss the issue label May 22, 2020
@leeherman99
Copy link
Author

leeherman99 commented May 22, 2020 via email

@janezd
Copy link
Contributor

janezd commented May 22, 2020

Heatmap still centers to zero if you choose one of diverging palettes (at least it should). The code is still there, we just eliminated the checkbox because it doesn't make sense to use a diverging palette unless you want to center, and it doesn't make sense to center without a diverging palette.

@leeherman99
Copy link
Author

Yes, but as I mentioned earlier, sometimes you want to center on a value other than zero (one, for example).

@janezd
Copy link
Contributor

janezd commented May 22, 2020

Yes, sure, I just wanted to answer that the code for centering is still there. :) We'll add parametrization if it doesn't complicate things. Sometimes a simple request can turn out to be ugly to implement.

@leeherman99
Copy link
Author

leeherman99 commented May 22, 2020 via email

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 a pull request may close this issue.

3 participants