aces / Loris

LORIS is a web-accessible database solution for longitudinal multi-site studies.
GNU General Public License v3.0
144 stars 173 forks source link

because BIDS : Disallow hyphens & underscores in VisitLabels etc. #7395

Open christinerogers opened 3 years ago

christinerogers commented 3 years ago

This issue (a) needs clarification in our docs
and (b) should get verified that it's well covered in our codebase - wherever the relevant variables are defined/created or loaded/validated

Because underscores and hyphens are delimiters in BIDS file paths, any variables in LORIS used in BIDS filepaths shouldn't contain them either, as we move increasingly into BIDS for more modalities.

This applies to anything in the BIDS path:

Comments on slack from cecile:

the `-` and `_` have meanings in BIDS, therefore they cannot be used in candidate ID, visit labels, scan type and so on
the `_` separates the different part of the file name and the `-` is used to separate the identifier from the value:
aka sub-<candidate>_ses-<visit>_<blablabla>.edf

Possible alternatives ? Develop a way of mapping visit labels, e.g. using another delimiter.

caught by @maltheism

christinerogers commented 3 years ago

include some recommendations:

christinerogers commented 1 year ago

@cmadjar any idea where to put this? it probably be easier to put this under the Visit labels troubleshooting guide or Readme ?

cmadjar commented 1 year ago

@christinerogers maybe both so the user cannot miss it? Or in the Readme add a link to the visit label guidelines?

christinerogers commented 1 year ago

@cmadjar sounds good - i'll self-assign and see if we can get this done simply during the release process.

christinerogers commented 1 month ago

this is still legit .