ENH: Add type annotations to pacify mypy

Author: effigies

nibabel/analyze.py

@@ -81,14 +81,17 @@
 can be loaded with and without a default flip, so the saved zoom will not
 constrain the affine.
 """
+from __future__ import annotations
+
+from typing import Type
 
 import numpy as np
 
 from .arrayproxy import ArrayProxy
 from .arraywriters import ArrayWriter, WriterError, get_slope_inter, make_array_writer
 from .batteryrunners import Report
 from .fileholders import copy_file_map
-from .spatialimages import HeaderDataError, HeaderTypeError, SpatialImage
+from .spatialimages import HeaderDataError, HeaderTypeError, SpatialHeader, SpatialImage
 from .volumeutils import (
     apply_read_scaling,
     array_from_file,
@@ -131,7 +134,7 @@
     ('glmax', 'i4'),
     ('glmin', 'i4'),
 ]
-data_history_dtd = [
+data_history_dtd: list[tuple[str, str] | tuple[str, str, tuple[int, ...]]] = [
     ('descrip', 'S80'),
     ('aux_file', 'S24'),
     ('orient', 'S1'),
@@ -172,7 +175,7 @@
 data_type_codes = make_dt_codes(_dtdefs)
 
 
-class AnalyzeHeader(LabeledWrapStruct):
+class AnalyzeHeader(LabeledWrapStruct, SpatialHeader):
     """Class for basic analyze header
 
     Implements zoom-only setting of affine transform, and no image
@@ -892,11 +895,11 @@ def may_contain_header(klass, binaryblock):
 class AnalyzeImage(SpatialImage):
     """Class for basic Analyze format image"""
 
-    header_class = AnalyzeHeader
+    header_class: Type[AnalyzeHeader] = AnalyzeHeader
     _meta_sniff_len = header_class.sizeof_hdr
-    files_types = (('image', '.img'), ('header', '.hdr'))
-    valid_exts = ('.img', '.hdr')
-    _compressed_suffixes = ('.gz', '.bz2', '.zst')
+    files_types: tuple[tuple[str, str], ...] = (('image', '.img'), ('header', '.hdr'))
+    valid_exts: tuple[str, ...] = ('.img', '.hdr')
+    _compressed_suffixes: tuple[str, ...] = ('.gz', '.bz2', '.zst')
 
     makeable = True
     rw = True

nibabel/benchmarks/bench_arrayproxy_slicing.py

@@ -26,7 +26,7 @@
 
 # if memory_profiler is installed, we get memory usage results
 try:
-    from memory_profiler import memory_usage
+    from memory_profiler import memory_usage  # type: ignore
 except ImportError:
     memory_usage = None
 

nibabel/brikhead.py

@@ -27,7 +27,6 @@
 am aware) always be >= 1. This permits sub-brick indexing common in AFNI
 programs (e.g., example4d+orig'[0]').
 """
-
 import os
 import re
 from copy import deepcopy

nibabel/casting.py

@@ -3,6 +3,7 @@
 Most routines work round some numpy oddities in floating point precision and
 casting.  Others work round numpy casting to and from python ints
 """
+from __future__ import annotations
 
 import warnings
 from numbers import Integral
@@ -110,7 +111,7 @@ def float_to_int(arr, int_type, nan2zero=True, infmax=False):
 
 
 # Cache range values
-_SHARED_RANGES = {}
+_SHARED_RANGES: dict[tuple[type, type], tuple[np.number, np.number]] = {}
 
 
 def shared_range(flt_type, int_type):

nibabel/cmdline/dicomfs.py

@@ -25,7 +25,7 @@ class dummy_fuse:
 
 
 try:
-    import fuse
+    import fuse  # type: ignore
 
     uid = os.getuid()
     gid = os.getgid()

nibabel/ecat.py

@@ -50,7 +50,7 @@
 
 from .arraywriters import make_array_writer
 from .fileslice import canonical_slicers, predict_shape, slice2outax
-from .spatialimages import SpatialImage
+from .spatialimages import SpatialHeader, SpatialImage
 from .volumeutils import array_from_file, make_dt_codes, native_code, swapped_code
 from .wrapstruct import WrapStruct
 
@@ -243,7 +243,7 @@
 patient_orient_neurological = [1, 3, 5, 7]
 
 
-class EcatHeader(WrapStruct):
+class EcatHeader(WrapStruct, SpatialHeader):
     """Class for basic Ecat PET header
 
     Sub-parts of standard Ecat File

nibabel/externals/netcdf.py

@@ -871,6 +871,7 @@ def __setattr__(self, attr, value):
             pass
         self.__dict__[attr] = value
 
+    @property
     def isrec(self):
         """Returns whether the variable has a record dimension or not.
 
@@ -881,16 +882,15 @@ def isrec(self):
 
         """
         return bool(self.data.shape) and not self._shape[0]
-    isrec = property(isrec)
 
+    @property
     def shape(self):
         """Returns the shape tuple of the data variable.
 
         This is a read-only attribute and can not be modified in the
         same manner of other numpy arrays.
         """
         return self.data.shape
-    shape = property(shape)
 
     def getValue(self):
         """

nibabel/filebasedimages.py

@@ -7,9 +7,11 @@
 #
 ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ##
 """Common interface for any image format--volume or surface, binary or xml."""
+from __future__ import annotations
 
 import io
 from copy import deepcopy
+from typing import Type
 from urllib import request
 
 from .fileholders import FileHolder
@@ -144,14 +146,14 @@ class FileBasedImage:
     work.
     """
 
-    header_class = FileBasedHeader
-    _meta_sniff_len = 0
-    files_types = (('image', None),)
-    valid_exts = ()
-    _compressed_suffixes = ()
+    header_class: Type[FileBasedHeader] = FileBasedHeader
+    _meta_sniff_len: int = 0
+    files_types: tuple[tuple[str, str | None], ...] = (('image', None),)
+    valid_exts: tuple[str, ...] = ()
+    _compressed_suffixes: tuple[str, ...] = ()
 
-    makeable = True  # Used in test code
-    rw = True  # Used in test code
+    makeable: bool = True  # Used in test code
+    rw: bool = True  # Used in test code
 
     def __init__(self, header=None, extra=None, file_map=None):
         """Initialize image

nibabel/freesurfer/mghformat.py

@@ -21,7 +21,7 @@
 from ..fileholders import FileHolder
 from ..filename_parser import _stringify_path
 from ..openers import ImageOpener
-from ..spatialimages import HeaderDataError, SpatialImage
+from ..spatialimages import HeaderDataError, SpatialHeader, SpatialImage
 from ..volumeutils import Recoder, array_from_file, array_to_file, endian_codes
 from ..wrapstruct import LabeledWrapStruct
 
@@ -87,7 +87,7 @@ class MGHError(Exception):
     """
 
 
-class MGHHeader(LabeledWrapStruct):
+class MGHHeader(LabeledWrapStruct, SpatialHeader):
     """Class for MGH format header
 
     The header also consists of the footer data which MGH places after the data

nibabel/gifti/gifti.py

@@ -11,10 +11,12 @@
 The Gifti specification was (at time of writing) available as a PDF download
 from http://www.nitrc.org/projects/gifti/
 """
+from __future__ import annotations
 
 import base64
 import sys
 import warnings
+from typing import Type
 
 import numpy as np
 
@@ -577,7 +579,7 @@ class GiftiImage(xml.XmlSerializable, SerializableImage):
     # The parser will in due course be a GiftiImageParser, but we can't set
     # that now, because it would result in a circular import.  We set it after
     # the class has been defined, at the end of the class definition.
-    parser = None
+    parser: Type[xml.XmlParser]
 
     def __init__(
         self,
@@ -832,17 +834,20 @@ def _to_xml_element(self):
             GIFTI.append(dar._to_xml_element())
         return GIFTI
 
-    def to_xml(self, enc='utf-8'):
+    def to_xml(self, enc='utf-8') -> bytes:
         """Return XML corresponding to image content"""
         header = b"""<?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE GIFTI SYSTEM "http://www.nitrc.org/frs/download.php/115/gifti.dtd">
 """
         return header + super().to_xml(enc)
 
     # Avoid the indirection of going through to_file_map
-    to_bytes = to_xml
+    def to_bytes(self, enc='utf-8'):
+        return self.to_xml(enc=enc)
 
-    def to_file_map(self, file_map=None):
+    to_bytes.__doc__ = SerializableImage.to_bytes.__doc__
+
+    def to_file_map(self, file_map=None, enc='utf-8'):
         """Save the current image to the specified file_map
 
         Parameters
@@ -858,7 +863,7 @@ def to_file_map(self, file_map=None):
         if file_map is None:
             file_map = self.file_map
         with file_map['image'].get_prepare_fileobj('wb') as f:
-            f.write(self.to_xml())
+            f.write(self.to_xml(enc=enc))
 
     @classmethod
     def from_file_map(klass, file_map, buffer_size=35000000, mmap=True):

nibabel/minc1.py

@@ -7,8 +7,10 @@
 #
 ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ### ##
 """Read MINC1 format images"""
+from __future__ import annotations
 
 from numbers import Integral
+from typing import Type
 
 import numpy as np
 
@@ -305,11 +307,11 @@ class Minc1Image(SpatialImage):
     load.
     """
 
-    header_class = Minc1Header
-    _meta_sniff_len = 4
-    valid_exts = ('.mnc',)
-    files_types = (('image', '.mnc'),)
-    _compressed_suffixes = ('.gz', '.bz2', '.zst')
+    header_class: Type[MincHeader] = Minc1Header
+    _meta_sniff_len: int = 4
+    valid_exts: tuple[str, ...] = ('.mnc',)
+    files_types: tuple[tuple[str, str], ...] = (('image', '.mnc'),)
+    _compressed_suffixes: tuple[str, ...] = ('.gz', '.bz2', '.zst')
 
     makeable = True
     rw = False

nibabel/minc2.py

@@ -155,7 +155,7 @@ class Minc2Image(Minc1Image):
     def from_file_map(klass, file_map, *, mmap=True, keep_file_open=None):
         # Import of h5py might take awhile for MPI-enabled builds
         # So we are importing it here "on demand"
-        import h5py
+        import h5py  # type: ignore
 
         holder = file_map['image']
         if holder.filename is None:

nibabel/nicom/dicomwrappers.py

@@ -127,8 +127,6 @@ class Wrapper:
     is_multiframe = False
     b_matrix = None
     q_vector = None
-    b_value = None
-    b_vector = None
 
     def __init__(self, dcm_data):
         """Initialize wrapper

nibabel/nifti1.py

@@ -10,8 +10,11 @@
 
 NIfTI1 format defined at http://nifti.nimh.nih.gov/nifti-1/
 """
+from __future__ import annotations
+
 import warnings
 from io import BytesIO
+from typing import Type
 
 import numpy as np
 import numpy.linalg as npl
@@ -87,8 +90,8 @@
 # datatypes not in analyze format, with codes
 if have_binary128():
     # Only enable 128 bit floats if we really have IEEE binary 128 longdoubles
-    _float128t = np.longdouble
-    _complex256t = np.longcomplex
+    _float128t: Type[np.generic] = np.longdouble
+    _complex256t: Type[np.generic] = np.longcomplex
 else:
     _float128t = np.void
     _complex256t = np.void
@@ -1814,7 +1817,7 @@ class Nifti1PairHeader(Nifti1Header):
 class Nifti1Pair(analyze.AnalyzeImage):
     """Class for NIfTI1 format image, header pair"""
 
-    header_class = Nifti1PairHeader
+    header_class: Type[Nifti1Header] = Nifti1PairHeader
     _meta_sniff_len = header_class.sizeof_hdr
     rw = True
 
@@ -1848,9 +1851,7 @@ def __init__(self, dataobj, affine, header=None, extra=None, file_map=None, dtyp
             self._affine2header()
 
     # Copy docstring
-    __init__.__doc__ = (
-        analyze.AnalyzeImage.__init__.__doc__
-        + """
+    __init__.__doc__ = f"""{analyze.AnalyzeImage.__init__.__doc__}
         Notes
         -----
 
@@ -1863,7 +1864,6 @@ def __init__(self, dataobj, affine, header=None, extra=None, file_map=None, dtyp
         :meth:`set_qform` methods can be used to update the codes after an image
         has been created - see those methods, and the :ref:`manual
         <default-sform-qform-codes>` for more details.  """
-    )
 
     def update_header(self):
         """Harmonize header with image data and affine

nibabel/openers.py

@@ -20,7 +20,7 @@
 
 # is indexed_gzip present and modern?
 try:
-    import indexed_gzip as igzip
+    import indexed_gzip as igzip  # type: ignore
 
     version = igzip.__version__
 

nibabel/parrec.py

@@ -1338,7 +1338,7 @@ def from_filename(
             strict_sort=strict_sort,
         )
 
-    load = from_filename
+    load = from_filename  # type: ignore
 
 
-load = PARRECImage.load
+load = PARRECImage.from_filename

nibabel/pkg_info.py

@@ -70,7 +70,7 @@ def cmp_pkg_version(version_str: str, pkg_version_str: str = __version__) -> int
     return _cmp(Version(version_str), Version(pkg_version_str))
 
 
-def pkg_commit_hash(pkg_path: str = None) -> tuple[str, str]:
+def pkg_commit_hash(pkg_path: str | None = None) -> tuple[str, str]:
     """Get short form of commit hash
 
     In this file is a variable called COMMIT_HASH. This contains a substitution
@@ -109,7 +109,7 @@ def pkg_commit_hash(pkg_path: str = None) -> tuple[str, str]:
         cwd=pkg_path,
     )
     if proc.stdout:
-        return 'repository', proc.stdout.strip()
+        return 'repository', proc.stdout.decode().strip()
     return '(none found)', '<not found>'