PennLINC / fw-heudiconv

Heuristic-based Data Curation on Flywheel
BSD 3-Clause "New" or "Revised" License
6 stars 11 forks source link

Can't tabulate dicom files on flywheel (gear or cli) #100

Open AnthonyRes opened 2 years ago

AnthonyRes commented 2 years ago

https://github.com/PennLINC/fw-heudiconv/blob/ad06d0e65196e815fd5e1d62bf66ed3008f058a5/fw_heudiconv/backend_funcs/query.py#L10

Description

I was trying to use fw-heudiconv-tabulate for a few of our projects to generate a seqinfo table, but the table being generated was empty. I believe it's because my files on flywheel are all dicoms. Line 78 checks if the file type of the acquisition is in CONVERTABLE_TYPES and skips the rest of the loop that would generate the table if it is not a convertable type, returning an empty array for acquisition_to_heudiconv(). Adding "dicom" to convertable types did generate the table.