Cyril Pernet
Cyril Pernet
yep that's what we discussed, if not then recommended (to avoid validation error) - thx @effigies
1) I think https://github.com/bids-standard/bids-specification/issues/1141 makes the point that modality is not well defined in BIDS, and as such is a separate issue of how OpenNeuro uses it. 2) I agree...
what was wrong? it runs for me on my windows laptop and on my linux server
I see something to do with parpool management ... until I fix that you can simply set N in [spmup_setparallel](https://github.com/CPernet/spmup/blob/master/utilities/spmup_setparallel.m) line 11, this should do the trick
good point we could use options to set N
maybe I fixed it https://github.com/CPernet/spmup/commit/40e827dbd346ba2403cdcd775d7082a4d9cdcca3
ooh that it bad -- works ok with mri_deface and others because there are still some face .. funny the affine transform founds a minimum with a tilt! got to...
we should then set that if field map then realign to 1st ; simple statement right
note I haven't fully tested all the options, in fact field map so far fail to produce good results - ie messed up that part
sounds very reasonable to separate preprocessing from analysis spmup_BIDS -- we can pass an input parameter that says to do the stats or not; if so it calls another function...