Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rdmfun transpose redundancy in rsanoiseceiling.m/py? #3

Open
jacob-prince opened this issue Apr 11, 2022 · 1 comment
Open

rdmfun transpose redundancy in rsanoiseceiling.m/py? #3

jacob-prince opened this issue Apr 11, 2022 · 1 comment
Assignees
Labels
question Further information is requested

Comments

@jacob-prince
Copy link
Collaborator

the default rdmfun behavior is to transpose a (voxel x condition) data matrix and then apply the pdist function to achieve a (condition x condition) RDM. however, in each monte carlo simulation, the signal and measurement variables already come in (condition x voxel) format, requiring each matrix to be transposed when passed to rdmfun() only to be re-transposed immediately afterward via rdmfun().

is there a reason for this double-transpose? if not, for clarity, we should consider removing both by modifying the line which sets the default rdmfun() and the line where rdmfun() is applied to signal and measurement.

@jacob-prince jacob-prince added the question Further information is requested label Apr 11, 2022
@kendrickkay
Copy link
Member

No real reason for the double transpose;it's just necessary for everytihng to work out. It just seems more natural for people to think in terms of "voxels x conditions x trials" as the standard format. Do you agree? If not, what do you suggest?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants