I'm trying to resolve a merge conflict in my fork. It looks like there may have been a change in expected syntax. Here is what is in the current AA master:
|
case 'phasediff' |
|
fmap.hdr = loadjson(f{1}); |
|
if isfield(fmap.hdr,'IntendedFor') |
|
fmap.hdr.session = get_taskname(sesspath,subjname,fmap.hdr.IntendedFor); |
|
else |
|
fmap.hdr.session = '*'; |
|
end |
|
fmap.fname = cellstr(spm_select('FPList',fullfile(sesspath,fieldmapDIR),[strrep(jfname,ftype,'') '.*.nii.gz'])); |
And here is my fork:
https://github.com/jooh/automaticanalysis/blob/efbff91382d605a07dd13ccbe5746a1b4702e97d/aa_engine/aas_processBIDS.m#L246-L253
It looks like the 'session' field now nests under the 'hdr' field. Is this the correct behaviour? Does anyone have an example BIDS script where IntendedFor is used, so we can test that this works as intended? Going to assign @jones-michael-s since that's where the change came from.
See also #187
I'm trying to resolve a merge conflict in my fork. It looks like there may have been a change in expected syntax. Here is what is in the current AA master:
automaticanalysis/aa_engine/aas_processBIDS.m
Lines 286 to 293 in 53ef81e
And here is my fork:
https://github.com/jooh/automaticanalysis/blob/efbff91382d605a07dd13ccbe5746a1b4702e97d/aa_engine/aas_processBIDS.m#L246-L253
It looks like the 'session' field now nests under the 'hdr' field. Is this the correct behaviour? Does anyone have an example BIDS script where IntendedFor is used, so we can test that this works as intended? Going to assign @jones-michael-s since that's where the change came from.
See also #187