added MedicalImagingData base class

extras/fileformats/extras/medimage/tests/test_deidentify.py

@@ -21,8 +21,12 @@ def dicom(request):
 
 def test_deidentify_dicom(dicom):
     assert str(dicom.metadata["PatientName"]) == "Doe^John"
+    assert dicom.metadata["InstitutionAddress"]
+    assert not dicom.metadata["PatientBirthDate"].endswith("0101")
     deidentified = dicom.deidentify()
     assert str(deidentified.metadata["PatientName"]) == "Anonymous^Anonymous"
+    assert deidentified.metadata["InstitutionAddress"] == ""
+    assert deidentified.metadata["PatientBirthDate"] == "19800101"
 
 
 def test_nifti_deidentify():

fileformats/medimage/__init__.py

@@ -1,5 +1,5 @@
 from ._version import __version__
-from .base import MedicalImage
+from .base import MedicalImagingData, MedicalImage
 
 
 from .misc import (
@@ -128,6 +128,7 @@
 
 __all__ = [
     "__version__",
+    "MedicalImagingData",
     "MedicalImage",
     "DicomImage",
     "Analyze",

fileformats/medimage/base.py

@@ -33,7 +33,25 @@
 )  # In Py<3.9 this is problematic "numpy.typing.NDArray[typing.Union[numpy.floating[typing.Any], numpy.integer[typing.Any]]]"
 
 
-class MedicalImage(WithClassifiers, FileSet):
+class MedicalImagingData(FileSet):
+    """Base class for all medical imaging data including pre-image raw data and
+    associated data"""
+
+    contains_phi: bool = True
+
+    @extra
+    def deidentify(
+        self,
+        out_dir: ty.Optional[Path] = None,
+        new_stem: ty.Optional[str] = None,
+        copy_mode: FileSet.CopyMode = FileSet.CopyMode.copy,
+    ) -> Self:
+        """Returns a new copy of the image with any subject-identifying information
+        stripped from the from the image header"""
+        raise NotImplementedError
+
+
+class MedicalImage(WithClassifiers, MedicalImagingData):
 
     INCLUDE_HDR_KEYS: ty.Optional[ty.Tuple[str, ...]] = None
     IGNORE_HDR_KEYS: ty.Optional[ty.Tuple[str, ...]] = None
@@ -42,7 +60,6 @@ class MedicalImage(WithClassifiers, FileSet):
     image_contents = ()
     allowed_classifiers = (ContentsClassifier,)
     exclusive_classifiers = (ImagingModality, AnatomicalEntity, Derivative)
-    contains_phi: bool = True
 
     @extra
     def read_array(self) -> DataArrayType:
@@ -63,14 +80,3 @@ def vox_sizes(self) -> ty.Tuple[float, float, float]:
     def dims(self) -> ty.Tuple[int, int, int]:
         """The dimensions of the image"""
         raise NotImplementedError
-
-    @extra
-    def deidentify(
-        self,
-        out_dir: ty.Optional[Path] = None,
-        new_stem: ty.Optional[str] = None,
-        copy_mode: FileSet.CopyMode = FileSet.CopyMode.copy,
-    ) -> Self:
-        """Returns a new copy of the image with any subject-identifying information
-        stripped from the from the image header"""
-        raise NotImplementedError

fileformats/medimage/raw/pet/base.py

@@ -2,6 +2,7 @@
 import typing as ty
 from pathlib import Path
 from fileformats.core import FileSet, extra
+from fileformats.medimage import MedicalImagingData
 from fileformats.generic import BinaryFile
 
 if sys.version_info >= (3, 12):
@@ -10,7 +11,7 @@
     from typing_extensions import Self
 
 
-class PetRawData(BinaryFile):
+class PetRawData(BinaryFile, MedicalImagingData):
     """Base class for raw PET data files"""
 
     @extra