Taylor Salo
Taylor Salo
For now, I recommend checking the sklearn version and modifying the parameter name based on that.
The changes look good to me. I just made a couple of suggestions to hopefully address the linter errors.
BIDS now supports arrays for RepetitionTimeExcitation and RepetitionTimePreparation, so I think all we need to do is allow the `cbv` suffix to use those fields.
Based on [this NeuroStars topic](https://neurostars.org/t/reuse-and-rerunning-with-fmriprep-25-2-2/34556), it looks like fMRIPrep is randomly selecting one of the echoes to process (or at least the metadata says as much), so I think this...
Can you make DICOMs and example data available for at least one type of fMRA sequence? I think that will help with getting it added to the spec and supported...
That is _a lot_! I think this will require more than a new suffix or allowing angio data in the func datatype. At least for this flavor, it looks like...
I'm not sure what makes the most sense. I suppose I was leaning toward focusing on structural images, and making it an initial step _before_ constructing the template, so that...
I think parameter estimate variance will be more useful than the RMSE spatial map, but I'm not sure they'll be a good substitute for R^2. I've tried to calculate R^2...
I have found that voxels with model fit failures end up having _terrible_ loglinear T2* estimates, so one thing I'd like to do that builds off this is to give...
Got the go-ahead to do this in the devs call today.