AuroreAA / ICSClust

What the Package Does in One 'Title Case' Line
GNU General Public License v3.0
0 stars 0 forks source link

General functions #4

Closed AuroreAA closed 1 year ago

AuroreAA commented 1 year ago

Do we want to have more general functions, for example tandemClust or ICSclust ? if yes, what do we want to provide as functionalities to the user with which amount of parameters?

klauschN commented 1 year ago

I would maybe prefer the functions ICSclust and PCAclust. And in general it would be nice to

  1. Choice of scatters
  2. Choice of clustering method
  3. Choice of component selection method
  4. Automatic Choice of number of cluster for given number

However looking at this it would require many many parameters to be passed on, or? Would it be then easier to have one general function with few parameters but does all more or less automatic and then we have the separate building block functions in case the user wants to "fine tune"?

aalfons commented 1 year ago

I generally agree that this will be nice to have when we release the package. It's not urgent for doing the simulations for our paper, where we need to do the steps separately to avoid repeated computations.

@klauschN, can you clarify what you mean by point 4? That is not clear to me.

Indeed, we certainly also need separate functions for separate steps at the user level, but this is what @AuroreAA has implemented now. I think having a broader function that does everything in one go will be nice, but there are some challenges.

klauschN commented 1 year ago

Ah typing error in 4. I thought that either the method gives the clustering for a specific number of clusters or is able to decide itself on the number of clusters.