search

AllenInstitute / transcriptomic_clustering

improved clustering pipeline for transcriptomic data
Other
4 stars 1 forks source link

Add merge module #82

Closed madiganz closed 3 years ago

madiganz commented 3 years ago

Problem: You would think that merging "merging" would be easy... Addressing #12 documentation

Solution: documented here http://confluence.corp.alleninstitute.org/display/IT/Merging

Validation: see test against tasic: test_merge_clusters

Performance on merging tasic data with chisq de method: in our python module: 441.535 sec in R: ~2 sec

MatthewAitken commented 3 years ago

It looks like tests are failing as well

sgratiy commented 3 years ago

I fixed the two tests related to merging. The other two failing tests have to do with cluster means and I expected that they should be fixed when we merge limma.