GMM covariance types examples overly complex / confusing
#10,863 opened on Mar 23, 2018
Repository metrics
- Stars
- (66,084 stars)
- PR merge metrics
- (Avg merge 10d) (90 merged PRs in 30d)
Description
I don't particularly like the example because it's supervised and uses a train/test split and identification with the original classes. I think it would be better to use a synthetic dataset and just show off the different covariance types.
It also fits the model on 4d data and only shows a 2d projection and that's not super intuitive imho.
Also, the example could be much simplified if we added a "get_covariance" function back to the model. I think we had that in the old GMM. Was there a reason not to add it to the new GMM? In many cases the user wants to be agnostic to the storage format of the covariance matrix, I think.