Tree plot for Hclust about statsplots.jl HOT 8 CLOSED

Oh, sorry, I just read your email again. Right now it would probably be even nicer to just keep it in your package. You could develop it an play with it, and it could always go to PlotRecipes later if broadly useful. Still willing to help getting the recipe right if you post a link.

from statsplots.jl.

That's very nice! IMHO the best place to have a recipe is always in the directory that defines the Type. That makes for the best dependency structure and ensures that the recipe is updated when the type or interface changes. It's an open question whether Clustering.jl will accept a dep on RecipesBase, though - though it's a safe (very light) dep.

from statsplots.jl.

@mkborregaard Got it. So maybe I should open an issue there, and ask whether they'd be willing to accept this? It would still need a lot of work though, and I think I'd need your help to get the combo working (though maybe the combo would go here since it depends on heatmap()?)

from statsplots.jl.

heatmap is not a problem. You could start by opening an issue and hear if they'd be willing to accept a PR. I'm happy to help as much as I can.

from statsplots.jl.

@mkborregaard Seems like the answer is no.

So then, is this the best place for it? I'm working on this as part of the development of a package for working with microbiome data. I could just leave it there for now, and move it elsewhere if it seems like it would be more broadly useful.

from statsplots.jl.

PlotRecipes is the best place for it, I think. There's a lot of functionality for graphs already, and one solution could be to add a general dendrogram layout to the graphplot recipe that did the dendrogram, and then just let the hclust recipe dispatch to that. If you're not up for that, just try making the pr marked WIP on PlotRecipes (in the misc.jl file) as it is, and we can discuss from there.

from statsplots.jl.

As we just talked about on EcoJulia/Microbiome.jl#19 it would be great to have this in StatPlots now, given that we're planning to let StatPlots try to support the types and operations in the Stats packages.

from statsplots.jl.

I'm just referencing this to Pull Request #214 where there has been an update on this.

from statsplots.jl.