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

maturity-index: make heatmap optional #126

Open
nbokulich opened this issue Oct 12, 2018 · 0 comments
Open

maturity-index: make heatmap optional #126

nbokulich opened this issue Oct 12, 2018 · 0 comments
Labels
good first issue Good for newcomers

Comments

@nbokulich
Copy link
Member

Improvement Description
Heatmaps are sooooo slooooow when there are many features (as is common). Waaay slower than the actual heavy lifting (supervised regression) in the maturity-index pipeline. Many users may not want to bother plotting these if they have other plotting methods in mind. This would also really speed up unit tests... Make optional?

Current Behavior
My hair turns grayer each time I need to wait for heatmaps to be produced by this plugin.

Proposed Behavior
Make heatmaps optional.

Questions

  1. How can these be made optional?
    • generate an empty index.html? (ugly)
    • produce a quick-and-dirty heatmap in seaborn (this is how the old viz did things and it was so much faster, but impenetrable to read when you have hundreds to thousands of features)
    • Wait for optional outputs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant