8000 Correctly handle units in all interfaces · Issue #786 · nipreps/fmriprep · GitHub
[go: up one dir, main page]

Skip to content
Correctly handle units in all interfaces #786
@effigies

Description

@effigies

Many of our interfaces that use img.header.get_zooms() assume mm+s units, rather than checking get_xyzt_units(), while the NIfTI standard also allows for meters, microns, milliseconds and microseconds. There will eventually be a dataset where this bites us, so we should try to address this before it becomes a real issue.

In the short term, it may make sense to create a helper function that takes an image and returns zooms in mm+s, and just call that instead of img.header.get_zooms() (not sure if this belongs more in fmriprep or niworkflows).

In the long term, I've proposed a new method on NIfTI headers that always returns mm+s (nipy/nibabel#567). We can use that as a template for a get_norm_zooms(img) function.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0