Merge pull request #528 from mgxd/fix/derivatives-dtype

FIX: Non-integer data coercion initialization
nipreps · May 27, 2020 · aea5805 · aea5805
2 parents e15694f + 28e0d4f
commit aea5805
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/niworkflows/interfaces/bids.py b/niworkflows/interfaces/bids.py
@@ -605,7 +605,7 @@ def _run_interface(self, runtime):
                         if np.issubdtype(data_dtype, np.integer):
                             new_data = np.rint(nii.dataobj).astype(data_dtype)
                         else:
-                            new_data = data_dtype(nii.dataobj)
+                            new_data = np.asanyarray(nii.dataobj, dtype=data_dtype)
                         # and set header to match
                         nii.set_data_dtype(data_dtype)
                         nii = nii.__class__(new_data, nii.affine, nii.header)

diff --git a/niworkflows/interfaces/tests/test_bids.py b/niworkflows/interfaces/tests/test_bids.py
@@ -276,7 +276,7 @@ def test_DerivativesDataSink_build_path(
     "space, size, units, xcodes, zipped, fixed, data_dtype",
     [
         ("T1w", (30, 30, 30, 10), ("mm", "sec"), (2, 2), True, [False], None),
-        ("T1w", (30, 30, 30, 10), ("mm", "sec"), (0, 2), True, [True], None),
+        ("T1w", (30, 30, 30, 10), ("mm", "sec"), (0, 2), True, [True], "float64"),
         ("T1w", (30, 30, 30, 10), ("mm", "sec"), (0, 0), True, [True], "<i4"),
         ("T1w", (30, 30, 30, 10), ("mm", None), (2, 2), True, [True], "<f4"),
         ("T1w", (30, 30, 30, 10), (None, None), (0, 2), True, [True], None),