Andrew Smith
Andrew Smith
Just pushed the latest joy. It's good! It works quite well. The test uses k-means initialization by default, and I fixed a bug in the covariance calculation that sealed the...
I went ahead and improved a load of other things we discussed. It's pretty solid now. Still occasionally fails, but steadily increasing the regularization constant on each failure usually fixed...
Cool. I didn't really do much in the way of a/b testing here. I'll see if I can tweak the parameters to get this to work better, or otherwise try...
I bounced back and forth between the `CholeskyFull` and `Diagonal` interpretations, and while they're both wrong it seems like the `CholeskyFull` has generally higher values for the variance, although it...
Can you break this down a little bit more, or cite reference literature or a reference implementation? I'm very invested in the stability of GMM, as it's the standard way...
Okay, do you want to push stuff to a separate branch on this project? I can also pull down changes and work on it then. I spent an hour or...
Sounds great. I'll work on the cholesky implementation first. It seems that's most of the numerical instability.
Cool. Why don't I take a crack at it first. I'm finding it pretty complicated to try and imagine things in non-log-space while looking at the sklearn code, so I'm...
I've taken a first crack at it with #155. I can't seem to figure out why the means aren't separating though. They do drift in the direction of the points,...
I would say that [`IndexGet` is the clearest](https://github.com/rust-lang/rfcs/pull/159/files#diff-8a993cf9991062c4937d16be43c7baefR52), because of the ease of implementing it for both `T` and `&T`. `IndexMove` is too restricted as a name, in that you...