search

carrien / free-speech

Analysis and plotting code for speech neuroimaging experiments.
MIT License
4 stars 5 forks source link

Wrapper function for get_gender, changes question to set expt.gender #18

Closed Comeani closed 3 years ago

Comeani commented 3 years ago

Want to change the question the experimenter is asked while setting up expt from the biological sex of the participant to their height, as VTL strongly correlates with body height.

Corresponding change to new run_EXPTNAME_expt scripts: if ~isfield(expt,'gender'), expt.gender = get_height; end

cwnaber commented 3 years ago

Is any other testing needed before this gets merged?

Comeani commented 3 years ago

Was the testing you did on an experiment running machine? If not I'll see if Lana can sanity check that and then I'll merge it.

cwnaber commented 3 years ago

Was the testing you did on an experiment running machine? If not I'll see if Lana can sanity check that and then I'll merge it.

No I just tested on my personal computer

lhantzsch commented 3 years ago

Tested it on Burnham, and is functioning as expected.

lhantzsch commented 3 years ago

When this gets merged, can you also update the run_modelExpt_expt.m to use this function?

cwnaber commented 3 years ago

Good call Lana, updated modelExpt on this branch so it'll merge alongside get_height.