nipy / heudiconv

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

Session template field not listed in default heuristic comment string #618

Open wasciutto opened 1 year ago

wasciutto commented 1 year ago

In the def infotodict(seqinfo) function provided by heudiconv's default heuristic file, it lists out the possible template fields:

"""Heuristic evaluator for determining which runs belong where

    allowed template fields - follow python string module:

    item: index within category
    subject: participant id
    seqitem: run number during scanning
    subindex: sub index within group
    """

However, it is missing the template field "session"

Summary

Platform details:

Choose one:

yarikoptic commented 1 year ago

I dont think we have a "default" one but we do have a good number of them probably having the same "problem":

❯ git grep 'Heuristic evaluator for determining which runs belong where'
heudiconv/heuristics/banda-bids.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/bids_ME.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/bids_PhoenixReport.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/bids_with_ses.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/cmrr_heuristic.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/convertall.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/example.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/multires_7Tbold.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/reproin.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/studyforrest_phase2.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/test_b0dwi_for_fmap.py:    """Heuristic evaluator for determining which runs belong where
heudiconv/heuristics/uc_bids.py:    """Heuristic evaluator for determining which runs belong where

interested to prepare PR to fix it up?