Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Output naming conventions #1491

Closed
KevinAquino opened this issue Feb 1, 2019 · 2 comments
Closed

Output naming conventions #1491

KevinAquino opened this issue Feb 1, 2019 · 2 comments
Labels
derivatives-rc1 Compliance with BIDS-Derivatives RC1 question

Comments

@KevinAquino
Copy link

Hi all,

We have been using fmriprep over two seperate versions and I have been noticing some names get changed e.g. the confounds file name gets changed to "confounds_regressors" or "probtissue" changing to "probseg". Does fmriprep spit out a mapping of names that can be called upon for different versions of fmriprep?

e.g.
GM_PROBTISSUE_SUFFIX = ......
CSF_PROBTSSUE_SUFFIX = ......

We are a writing a little toolbox that uses fmriprep outputs and the naming conventions changing is hard to future proof the code. Its been by other people in my lab so I'm not sure if its because of different inputs (or incorrect spec. of input flags -- it doesn't look like it though). I could create one for our plug in, but it would be great if it already existed! :)

@effigies effigies added the derivatives-rc1 Compliance with BIDS-Derivatives RC1 label Feb 1, 2019
@effigies
Copy link
Member

effigies commented Feb 1, 2019

The changing conventions are because of the recent (August 2018) work on BIDS derivatives, which resulted in larger changes than any of us anticipated, going in.

That specification is in release-candidate mode, which means that only very minor changes will be accepted before being released as 1.0, mostly where it's found to be internally inconsistent. (After 1.0, no backwards-incompatible changes will be accepted until 2.0.)

The upshot is that our naming conventions should not be changing again. We don't have a name mapping like you describe, though that certainly would have been nice. If you need a fairly concise summary of the changes as the affect fMRIPrep, you can look through #1325 and #1333.

@KevinAquino
Copy link
Author

Ok cool, thanks Chris thats very helpful!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
derivatives-rc1 Compliance with BIDS-Derivatives RC1 question
Projects
None yet
Development

No branches or pull requests

3 participants