Merge remote-tracking branch 'upstream/master' into correct-bug-OrthoSlicer3D

Author: effigies

.coveragerc

@@ -1,9 +1,18 @@
 [run]
 branch = True
 source = nibabel
-include = */nibabel/*
 omit =
     */externals/*
     */benchmarks/*
-    */tests/*
     nibabel/_version.py
+
+[report]
+exclude_also =
+    def __repr__
+    if (ty\.|typing\.)?TYPE_CHECKING:
+    class .*\((ty\.|typing\.)Protocol\):
+    @(ty\.|typing\.)overload
+    if 0:
+    if __name__ == .__main__.:
+    @(abc\.)?abstractmethod
+    raise NotImplementedError

.github/workflows/test.yml

@@ -125,9 +125,9 @@ jobs:
           - os: ubuntu-latest
             python-version: 3.8
             dependencies: 'min'
-          # NumPy 2.0
+          # NoGIL
           - os: ubuntu-latest
-            python-version: '3.12'
+            python-version: '3.13-dev'
             dependencies: 'dev'
         exclude:
           # x86 for Windows + Python<3.12
@@ -168,11 +168,18 @@ jobs:
           submodules: recursive
           fetch-depth: 0
       - name: Set up Python ${{ matrix.python-version }}
+        if: "!endsWith(matrix.python-version, '-dev')"
         uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
           architecture: ${{ matrix.architecture }}
           allow-prereleases: true
+      - name: Set up Python ${{ matrix.python-version }}
+        if: endsWith(matrix.python-version, '-dev')
+        uses: deadsnakes/action@v3.1.0
+        with:
+          python-version: ${{ matrix.python-version }}
+          nogil: true
       - name: Display Python version
         run: python -c "import sys; print(sys.version)"
       - name: Install tox
@@ -182,7 +189,7 @@ jobs:
       - name: Show tox config
         run: tox c
       - name: Run tox
-        run: tox -v --exit-and-dump-after 1200
+        run: tox -vv --exit-and-dump-after 1200
       - uses: codecov/codecov-action@v4
         if: ${{ always() }}
         with:

.pre-commit-config.yaml

@@ -16,7 +16,7 @@ repos:
     rev: v0.3.4
     hooks:
       - id: ruff
-        args: [--fix, --show-fix, --exit-non-zero-on-fix]
+        args: [--fix, --show-fixes, --exit-non-zero-on-fix]
         exclude: = ["doc", "tools"]
       - id: ruff-format
         exclude: = ["doc", "tools"]

Changelog

@@ -25,6 +25,33 @@ Eric Larson (EL), Demian Wassermann, Stephan Gerhard and Ross Markello (RM).
 
 References like "pr/298" refer to github pull request numbers.
 
+Upcoming release (To be determined)
+===================================
+
+New features
+------------
+
+Enhancements
+------------
+ * Ability to read data from many multiframe DICOM files that previously generated errors
+
+Bug fixes
+---------
+ * Fixed multiframe DICOM issue where data could be flipped along slice dimension relative to the
+   affine
+ * Fixed multiframe DICOM issue where ``image_position`` and the translation component in the
+   ``affine`` could be incorrect
+
+Documentation
+-------------
+
+Maintenance
+-----------
+
+API changes and deprecations
+----------------------------
+
+
 5.2.1 (Monday 26 February 2024)
 ===============================
 

doc/source/conf.py

@@ -28,6 +28,10 @@
     import tomli as tomllib
 
 # Check for external Sphinx extensions we depend on
+try:
+    import numpy as np
+except ImportError:
+    raise RuntimeError('Need to install "numpy" package for doc build')
 try:
     import numpydoc
 except ImportError:
@@ -45,6 +49,11 @@
         'Need nibabel on Python PATH; consider "make htmldoc" from nibabel root directory'
     )
 
+from packaging.version import Version
+
+if Version(np.__version__) >= Version('1.22'):
+    np.set_printoptions(legacy='1.21')
+
 # -- General configuration ----------------------------------------------------
 
 # We load the nibabel release info into a dict by explicit execution

doc/tools/build_modref_templates.py

@@ -9,7 +9,7 @@
 import sys
 
 # version comparison
-from distutils.version import LooseVersion as V
+from packaging.version import Version as V
 from os.path import join as pjoin
 
 # local imports
@@ -73,6 +73,8 @@ def abort(error):
                 if re.match('^_version_(major|minor|micro|extra)', v)
             ]
         )
+
+    source_version = V(source_version)
     print('***', source_version)
 
     if source_version != installed_version:

nibabel/_compression.py

@@ -17,7 +17,7 @@
 
 from .optpkg import optional_package
 
-if ty.TYPE_CHECKING:  # pragma: no cover
+if ty.TYPE_CHECKING:
     import indexed_gzip  # type: ignore[import]
     import pyzstd
 

nibabel/arrayproxy.py

@@ -57,7 +57,7 @@
 KEEP_FILE_OPEN_DEFAULT = False
 
 
-if ty.TYPE_CHECKING:  # pragma: no cover
+if ty.TYPE_CHECKING:
     import numpy.typing as npt
     from typing_extensions import Self  # PY310
 
@@ -75,19 +75,17 @@ class ArrayLike(ty.Protocol):
     shape: tuple[int, ...]
 
     @property
-    def ndim(self) -> int: ...  # pragma: no cover
+    def ndim(self) -> int: ...
 
     # If no dtype is passed, any dtype might be returned, depending on the array-like
     @ty.overload
-    def __array__(
-        self, dtype: None = ..., /
-    ) -> np.ndarray[ty.Any, np.dtype[ty.Any]]: ...  # pragma: no cover
+    def __array__(self, dtype: None = ..., /) -> np.ndarray[ty.Any, np.dtype[ty.Any]]: ...
 
     # Any dtype might be passed, and *that* dtype must be returned
     @ty.overload
-    def __array__(self, dtype: _DType, /) -> np.ndarray[ty.Any, _DType]: ...  # pragma: no cover
+    def __array__(self, dtype: _DType, /) -> np.ndarray[ty.Any, _DType]: ...
 
-    def __getitem__(self, key, /) -> npt.NDArray: ...  # pragma: no cover
+    def __getitem__(self, key, /) -> npt.NDArray: ...
 
 
 class ArrayProxy(ArrayLike):

nibabel/brikhead.py

@@ -391,7 +391,7 @@ def get_affine(self):
         # AFNI default is RAI- == LPS+ == DICOM order.  We need to flip RA sign
         # to align with nibabel RAS+ system
         affine = np.asarray(self.info['IJK_TO_DICOM_REAL']).reshape(3, 4)
-        affine = np.row_stack((affine * [[-1], [-1], [1]], [0, 0, 0, 1]))
+        affine = np.vstack((affine * [[-1], [-1], [1]], [0, 0, 0, 1]))
         return affine
 
     def get_data_scaling(self):

nibabel/cmdline/dicomfs.py

@@ -37,7 +37,7 @@ class dummy_fuse:
 import nibabel as nib
 import nibabel.dft as dft
 
-encoding = locale.getdefaultlocale()[1]
+encoding = locale.getlocale()[1]
 
 fuse.fuse_python_api = (0, 2)