mnc2nii do not populate time_units header field in the NIfTI file about minc-tools

[gdevenyi]

I can find no documentation regarding what values are valid for time_units in the NIFTI library...

[gdevenyi]

After falling down the terrifying rabbithole that is the NIFTI1 library...
https://github.com/NIFTI-Imaging/nifti_clib/blob/9659cbecc827e15c07ccc1a03023e8c348aa14fe/nifti2/nifti1.h#L1329-L1333

[gdevenyi]

We need

nim->time_units = NIFTI_UNITS_SEC ;

Or similar set.

Need to find the docs on the units inside MINC.

[gdevenyi]

There is a units specification for dimensions:

               time:units = "s" 

However,

s - String that specifies the units of a dimension, if applicable. The units should be compatible with the UCAR udunits library. In MINC 2.0, this attribute is generally included for informational purposes only and is not interpreted by the MINC library code.

So we either assume seconds, or use this to check and set appropriately on conversion.

[cmadjar]

This issue was indeed resolved by the fix. Sorry, I forgot to update and close the issue...