nipy / heudiconv

Flexible DICOM conversion into structured directory layouts
https://heudiconv.readthedocs.io
Other
234 stars 125 forks source link

anonymized IDs aren't BIDSified; underscores cause later problems #247

Open toddt opened 6 years ago

toddt commented 6 years ago

I have an anonymizer that is a simple lookup from a two column file: SAX_EIB_034, SAX_OA_34

When I run heudiconv on this, it correctly changes SAX_EIB_034 to sub-SAXEIB034 appropriately, but doesn't run the same correcting on the SAX_OA_34 string. This then causes files like the sub_SAXEIB034_scans.tsv to be created as sub-SAX_scans.tsv in the anonymized directory.

mgxd commented 6 years ago

this is something to consider, but if you are defining the new anonymized subject id - why not just make it bids compliant?

toddt commented 6 years ago

I did, of course. I just wanted to report that this might not be behaving as desired.