Merge remote-tracking branch 'krishs/master' into main-master

* krishs/master:
  BF: fixed 3 bugs

Author: matthew-brett

nibabel/freesurfer/mghformat.py

@@ -129,22 +129,11 @@ def __str__(self):
 
     def __getitem__(self, item):
         ''' Return values from header data
-
-        Examples
-        --------
-        >>> hdr = MGHHeader()
-        >>> hdr['type'] == 1
-        True
         '''
         return self._header_data[item]
 
     def __setitem__(self, item, value):
         ''' Set values in header data
-
-        Examples
-        --------
-        >>> hdr = MGHHeader()
-        >>> hdr['type'] = 1
         '''
         self._header_data[item] = value
 
@@ -488,7 +477,9 @@ def to_file_map(self, file_map=None):
         self._write_header(mghf, hdr)
         self._write_data(mghf, data, hdr)
         self._write_footer(mghf, hdr)
-        mghf.close()
+        # if the file_map points to a filename, close it
+        if file_map['image'].fileobj is None:  # was filename
+            mghf.close()
         self._header = hdr
         self.file_map = file_map
 

nibabel/tests/test_files_interface.py

@@ -12,7 +12,7 @@
 
 import numpy as np
 
-from .. import class_map, Nifti1Image, Nifti1Pair
+from .. import class_map, Nifti1Image, Nifti1Pair, MGHImage
 from ..py3k import BytesIO
 from ..fileholders import FileHolderError
 
@@ -32,7 +32,12 @@ def test_files_images():
             assert_equal(value.filename, None)
             assert_equal(value.fileobj, None)
             assert_equal(value.pos, 0)
-        img = klass(arr, aff)
+        # MGHImage accepts only a few datatypes
+        # so we force a type change to float32
+        if klass == MGHImage:
+            img = klass(arr.astype(np.float32), aff)
+        else:
+            img = klass(arr, aff)
         for key, value in img.file_map.items():
             assert_equal(value.filename, None)
             assert_equal(value.fileobj, None)