-
Notifications
You must be signed in to change notification settings - Fork 56
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
Manually adjust correlation limits in plots #27
Comments
Hi, Do you have an example use case? If I understand correctly, the suggestion is to allow the user to show a range of correlations other than -1 to 1. I think that the issue of allowing users to change how the color scales are applied to the displayed values is interesting. I have questions about allowing users to choose the range of correlations to display. My intuition is that removing the weakest correlations is the most useful use case of giving the user to a broad ability to choose the correlation limits. I think there's one problematic scenario in particular with user-supplied limits: the user chooses to show moderate correlations while ignoring weak and strong relationships. As a viewer of the plot, it seems weird that a plot might show correlations between 0.3 and 0.6 and then it would be left to the viewer to reason or guess whether the missing relationships are weaker (i.e.< 0.3) or stronger (i.e. >0.6) than the relationships shown. -daryn |
Those are great points, @thisisdaryn. Let's close this issue and leave the plot limits as they are for now. We can revisit this issue in the future if users open new issues asking about this. |
This issue has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue. |
Currently, network_plot() and rplot() limit the colours and alpha etc to -1 and 1. However, with the addition of operations, these limits can be escaped.
Furthermore, it might just be nice for users to adjust these limits to suit their own purpose.
Thanks to @jcberny for the suggestion.
The text was updated successfully, but these errors were encountered: