Description

BIDS2Legacy crashes for Philips_PCASL_2DEPI_3.2.2.1_2 because it doesn't convert anything from BIDS.

• The issue is the name 'EMI_301' - the '_' is lost during the process...
• Dcm2niix still creates 'EMI_301' in temp.
• In rawdata, I already have 'sub_EMI301' and not 'sub_EMI_301' so later, BIDS2Legacy relies on information from sourcedata (wrongly) and doesn't find the subject in rawdata.
• We need to skip the _ and replace by an empty character in these places:
  • reading subject structure
  • DCM2NII
  • NII2BIDS
• Function xASL_imp_CheckForAliasInVisit inside xASL_imp_NII2BIDS_Subject seems to be doing that for me.
• At line 92, subjectName is still EMI_301, but bidsLabel.subject = xASL_adm_CorrectName shortens it to EMI301.

What I propose is that we need to remove all _ in patient names systematically and also in session/visit names as these are also in BIDS.

Release notes

Improve import modularity and BIDS format of subject IDs.