mlampros / ClusterR

Gaussian mixture models, k-means, mini-batch-kmeans and k-medoids clustering
https://mlampros.github.io/ClusterR/
84 stars 29 forks source link

[Fix #40] Don't class the output of Optimal_Clusters_GMM #41

Closed vspinu closed 1 year ago

vspinu commented 1 year ago

As per https://github.com/mlampros/ClusterR/issues/40#issuecomment-1346175663

mlampros commented 1 year ago

this PR closes #40 Yes compared to the other 2 functions "Optimal_Clusters_KMeans()" and "Optimal_Clusters_Medoids()" I added the class 'Gaussian Mixture Models' which is actually not required I'll merge this PR (although checks were not successful) and I'll modify the tests that throw an error related to the 'Gaussian Mixture Models' class

vspinu commented 1 year ago

Arh, forgot to test before submitting :crying_cat_face:

mlampros commented 1 year ago

everything is ok, I modified the tests and pushed the changes. I'll wait a couple of weeks to submit the new version to CRAN because I submitted almost a week ago