-
-
Notifications
You must be signed in to change notification settings - Fork 101
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
Feature request: autosize #30
Comments
Cool idea! |
I've implemented a solution to dynamically adjust the heatmap size based on the number of features in the dataset. The figure size scales proportionally, ensuring all features are displayed clearly, regardless of dataset size. Would you like me to contribute this change to the project? Below are the before and after visuals for your reference: This resolves the issue of overcrowded plots for larger datasets. |
hey @Swish78 , i like the simplicity of it 💪 the only caveat here is that it doesn't take into account the length of the numbers displayed in the heat-map. I guess I'm brainstorming here with you - in the (rare?) case where a user wants use |
Hi @shakedzy, Thanks for your feedback! My initial approach was to consider both the precision of the numbers and the number of features when adjusting the figure size. However, I noticed this led to increased memory consumption, which might not be ideal, especially for larger plots. That said, I assumed most users typically stick to a .2f format, which is why I leaned toward a simpler feature-based resizing. However, if memory consumption isn't a major concern, we could definitely account for both the number of features and the precision in the figure size configuration. Let me know your thoughts! |
yeah, that makes sense.. I think this solution seems great, i like it, and it's an improvement for the current state anyhow, so i think we'll go with it. Thanks! |
Released in version 0.7.8 |
It's easy to adjust figsize yourself, and you can pass it to associations(), but it would be nice if this method could estimate an appropriate figsize based on the number of columns.
The text was updated successfully, but these errors were encountered: