m-Py / anticlust

Subset partitioning via anticlustering
Other
29 stars 5 forks source link

Remove argument `parallelize` from `anticlustering()` #35

Closed m-Py closed 5 years ago

m-Py commented 5 years ago

As the exchange method is now the default algorithm (and is very strongly recommended in comparison to random sampling) it seems a bit too much to include a parallelize option for random sampling -- remove it. This also means removing the argument seed which is only relevant for making parallel random sampling reproducible. Removing two arguments from the anticlustering() is good because it has too many right now, and this change will clean up the code base in general.

m-Py commented 5 years ago

see 41b9d7e