Possible issue in pspm_cfg_run_glm_hp_rew

[LouiseRoseman]

Hi,
I’m having an issue running the HP reward-conditioning GLM in PsPM 7.0.0.
I get the following error:
Dot indexing is not supported for variables of this type.
Error in pspm_cfg_run_glm_hp_rew (line 12)
model = params.model;
I’ve traced this to pspm_cfg_run_glm_hp_rew.m, which calls:
params = pspm_cfg_run_glm(job, def_filter);
model = params.model;
options = params.options;
However, in my PsPM 7.0.0 version, pspm_cfg_run_glm.m returns:
out = {model.modelfile};
rather than a structure containing params.model and params.options. So I think this is the issue?
Could anyone advise on how to fix it?
Many thanks!

[LouiseRoseman]

I changed the function to:

`
%{
%original:
function out = pspm_cfg_run_glm_hp_rew(job)
% Updated on 08-01-2024 by Teddy
global settings
if isempty(settings), pspm_init; end
% set modality
modality = 'hp';
modelspec = 'hp_rew';
f = strcmpi({settings.glm.modelspec}, modelspec);
def_filter = settings.glm(f).filter;
params = pspm_cfg_run_glm(job, def_filter);
% get parameters
model = params.model;
options = params.options;
% basis function
model.bf.fhandle = str2func('pspm_bf_hprf_rew');
model.bf.args = [];
model.modality = modality;
model.modelspec = modelspec;
pspm_glm(model, options);
out = {model.modelfile};
%}

% new LS edited:
function out = pspm_cfg_run_glm_hp_rew(job)

global settings
if isempty(settings), pspm_init; end
% initialise
model = struct();
options = struct();
% set modality
model.modality = 'hp';
model.modelspec = 'hp_rew';

f = strcmpi({settings.glm.modelspec}, model.modelspec);
def_filter = settings.glm(f).filter;

% basis function
model.bf.fhandle = str2func('pspm_bf_hprf_rew');
model.bf.args = [];

out = pspm_cfg_run_glm(job, model, options);
`

Based on the similar functions but would appreciate confirmation if this is right. Thank you :)

[4gwe]

Thanks. I’ll take a look. Appreciate your help.

[dominikbach]

Thanks for posting this - see issue #919
This is fixed now in the development branch with pull request #918.