bioFAM / MOFA

Multi-Omics Factor Analysis
GNU Lesser General Public License v3.0
231 stars 57 forks source link

Missing for loop #35

Closed vsomnath closed 5 years ago

vsomnath commented 5 years ago

https://github.com/bioFAM/MOFA/blob/c20929be1ba5659fce9d93780071501962790934/mofapy/core/distributions.py#L168

In the above referenced line, it looks like a for-loop is missing, since the variable n is used but not defined within the function.

rargelaguet commented 5 years ago

This is correct, thanks for the heads up! But just to clarify, currently we don't use multivariate normals so this line should not intervene at all .

vsomnath commented 5 years ago

Thanks for the quick response, and the clarification!