brianhie / trajectorama

Multi-study integration of cellular trajectories
http://trajectorama.csail.mit.edu
MIT License
18 stars 3 forks source link

X_nonzero centering #1

Closed frenkiboy closed 5 years ago

frenkiboy commented 5 years ago

https://github.com/brianhie/coscape/blob/cee22438fafe10cebc0b0b4e4d9d1ee322825d2e/bin/utils.py#L40

Am not 100% certain, but should X_nonzero be centered before calculating variance?

Cheers,

v

brianhie commented 5 years ago

Thanks for the heads up -- that does appear to be a calculation error :\

brianhie commented 5 years ago

fixed in https://github.com/brianhie/coscape/pull/2