Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update cfmm_bruker heuristic #43

Merged
merged 2 commits into from
Aug 20, 2024
Merged

update cfmm_bruker heuristic #43

merged 2 commits into from
Aug 20, 2024

Conversation

AlanKuurstra
Copy link
Contributor

forked heudiconv to add capability of:

  • handling dicom datetimes with timezone info
  • adding dicom metadata to .json sidecar when using a single n-dimensional dicom, when n>=4

changed tar2bids to:

  • add blockEPI acquisition to bruker heuristic
  • modify bruker heuristic to add bruker specific parameters to the .json sidecar
  • use forked version of heudiconv (required for bruker data)

@akhanf
Copy link
Member

akhanf commented Aug 19, 2024

great! I'm building the container to test, will be available in docker://khanlab/tar2bids:bruker when it is ready -- I'd like to give it a spin myself on some marm data acquired recently..

@akhanf
Copy link
Member

akhanf commented Aug 20, 2024

Tried out the docker container on one of my marmoset test cases and worked great. I also made a minor fix to the MP2RAGE and DTI parts of the heuristic, and committed that to the branch I built the container on. Will merge that in.

Thanks!

Copy link
Member

@akhanf akhanf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, and container builds and runs correctly.

MP2RAGE: had wrong series for UNI and inversions, and T1map not present. Raw
complex data removed from heuristics (series XX0003) since was not
converting into 3d volumes properly..  Note: this assumes XX0001 is
always the inversions, and XX0002 are the UNI images.

DTI: made case-insensitive
@akhanf akhanf added the bug Something isn't working label Aug 20, 2024
@akhanf akhanf merged commit bb1485d into khanlab:master Aug 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants