Deprecate RSA functions
PhilippKaniuth opened this issue · 0 comments
PhilippKaniuth commented
There is a well-maintained reference implementation for Representational Similarity Analysis, the rsatoolbox
. I therefore suggest to deprecate THINGSvision
's RSA functions.
Advantages:
- End users don't get confused as they only have to use a single gold-standard package for DNN feature extraction and one for RSA analyses.
THINGSvision
avoids getting issues raised asking for RSA functions currently not implemented inTHINGSvision
, e.g. other distance functions.- Less code for
THINGSvision
to maintain as the focus is solely on feature extraction.
Possibly PR THINGSvision
's RSA functions to the rsatoolbox
if they turn out to be more efficient.