-
Notifications
You must be signed in to change notification settings - Fork 33
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
View_corr_mat() - make new || leave old #149
Comments
Thanks @wingedRuslan - can you add an example of the additional functions that seaborn heatmap gives us? I personally prefer the current version because I like the black line around the outside of the plot, and I can't think of options that heatmap will give us that are useful in this case. Happy to be wrong though! |
@KirstieJane, if the black line around the outside of the plot is the only reason, I could add this feature to the new function as well :) you did this with the following line of code
|
@KirstieJane, the only difference (that we care about) could be concerning the improved colorbar:
|
Link to the existed function - |
Thanks for the link @wingedRuslan! I like that function actually 😄 This is the code I was thinking of that's at the top of the tutorials:
The correct way to threshold the graph is using the To be clear - we do not need to pass a graph The So the current code looks great. Lets add Thanks for checking! |
Heya!
This issue is opened to decide what to do with the
view_corr_mat
and partially to address the issue #148.view_corr_mat
inBasically, it is almost identical function to one in scripts/visualisation_commands.py. The one in visualisation_commands.py is better because PR #108 updated the view_corr_mat in visualisation_commands.py, now the function accepts corr_mat as a dataframe object, numpy_array, and as a path to file containing corr_mat.
What I suggested is to redesign
view_corr_mat
in order to use theseaborn.heatmap()
.Why? - from my perspective, it is better to rely on the already created solutions and seaborn.heatmap has a wide variety of parameters so it provides great flexibility to adjust plot to your preferences.
seaborn docs
Here are the comparisons (saved viz outputs) between the existed function and the potentially new one:
Existed function default call:
Seaborn recreate the same plot:
Use new colormap - 'YlGnBu'
To sum up, with seaborn.heatmat we could have the same figures as with the existed
corr_mat
but as a bonus a lot more nice features (read - parameters to function).The text was updated successfully, but these errors were encountered: