Fixed conversion to MixtureModel from :diag

davidavdav / GaussianMixtures.jl

Large scale Gaussian Mixture Models

Other

95 stars 36 forks source link

Closed JHonaker closed 3 years ago

JHonaker commented 3 years ago

The square root was not properly broadcast along the diagonal covariance when converting from a diagonal GMM to a MixtureModel from Distributions.jl.

davidavdav commented 3 years ago

I thought I'd seen this before, but I'm sure github knows better,

JHonaker commented 3 years ago

The last commit added the square root, but didn’t broadcast it.