Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@m-marqx
Copy link
Contributor

@m-marqx m-marqx commented Jul 25, 2023

This pull request will add an example of corr_interactive_plot in readme.md and /examples directories.


After posting this example, I noticed one thing: when the user sets the split values with "pos" or "neg" the usual method to calculate the color scale plots different colors compared to corr_plot, this is a problem? If it's an issue, I can add the following code after calculating vmax and vmin in corr_interactive_plot:

vmax = -vmin if split == "neg" else vmax
vmin = -vmax if split == "pos" else vmin

This fix should address the color problem.

akanz1
akanz1 previously approved these changes Jul 26, 2023
Copy link
Owner

@akanz1 akanz1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you for the update. I leave it up to you to decide about the colors, whichever you think is more suitable. I'll run and test everything before creating a release so if it bothers me then i can still make a change :)

@m-marqx
Copy link
Contributor Author

m-marqx commented Jul 26, 2023

I've fixed the colors and updated the readme.md, I think this fix would be more suitable if the color scale matches corr_plot, because the main idea behind corr_interactive_plot is to be very similar to corr_plot but with interactive features.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@akanz1 akanz1 merged commit 5bb6913 into akanz1:main Jul 27, 2023
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.

2 participants