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

prepare_fullDCM throws an error when using only Gaussian sources. #20

Closed
SRSteinkamp opened this issue Dec 8, 2016 · 1 comment
Closed
Labels

Comments

@SRSteinkamp
Copy link

Dear all,

I found that, when using only Gaussian source in prepare_fullDCM, line 99 throws an error, as (1:sum([sources.type]~=0)) returns an empty 1x0 matrix:

inG.indr = inG.ind2(end) + (1:sum([sources.type]~=0));

Additionally, there will be no unique offset parameters for other Gaussian observational sources.

Best,
Simon

@lionel-rigoux
Copy link
Member

Originally, the decoding function was supposed to be:

  • a softmax for binary response, with one parameter (the temperature)
  • the identity for gaussian responses (no parameters)

However, 1) the temperature is correlated with the behavioural mapping parameters, and 2) for all sources we need an additional parameter to offset the behavioural response by an offset.

The new default will then be set to the number of sources minus one (source number one being the BOLD response).

For more complex observation functions, inG.indr and dim.n_phi need to be set by hand by the user.

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

No branches or pull requests

2 participants