qiime2 / q2-longitudinal

QIIME 2 plugin for paired sample comparisons
BSD 3-Clause "New" or "Revised" License
9 stars 18 forks source link

maturity-index: make heatmap optional #126

Open nbokulich opened 5 years ago

nbokulich commented 5 years ago

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.