From e44b8ba2696c97765f6fe9f90a8cfda34f69cfdf Mon Sep 17 00:00:00 2001
From: Erik O Gabrielsson <erik.o.gabrielsson@sectra.com>
Date: Wed, 29 Nov 2023 21:16:02 +0100
Subject: [PATCH 1/2] Parse philips tiff mettadata

---
 tiffslide/tiffslide.py | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/tiffslide/tiffslide.py b/tiffslide/tiffslide.py
index 6120a98..8d89fe2 100644
--- a/tiffslide/tiffslide.py
+++ b/tiffslide/tiffslide.py
@@ -13,6 +13,8 @@
 from typing import Iterator
 from typing import Literal
 from typing import Mapping
+from typing import Optional
+from typing import Tuple
 from typing import TypeVar
 from typing import overload
 from warnings import warn
@@ -644,6 +646,7 @@ class _PropertyParser:
         scn="leica",
         bif="ventana",
         ndpi="hamamatsu",
+        philips="philips_tiff"
         # add more when needed
     )
 
@@ -843,6 +846,37 @@ def parse_generic_tiff(self) -> dict[str, Any]:
         md.update(self.collect_level_info(series0))
         return md
 
+    def parse_philips_tiff(self) -> dict[str, Any]:
+        """parse Philips tiff properties"""
+        md = self.parse_generic_tiff()
+        if self._tf.philips_metadata is None:
+            return md
+        philips_metadata = ElementTree.fromstring(self._tf.philips_metadata)
+        def get_first_attribute_with_name(
+            root: ElementTree.Element,
+            name: str
+        ) -> Optional[str]:
+            return next(
+                root.iterfind(f".//Attribute[@Name='{name}']")
+            ).text
+        md[PROPERTY_NAME_VENDOR] = get_first_attribute_with_name(
+            philips_metadata,
+            'DICOM_MANUFACTURER'
+        )
+        pixel_spacing_attribute = get_first_attribute_with_name(
+            philips_metadata,
+            'DICOM_PIXEL_SPACING'
+        )
+        if pixel_spacing_attribute is not None:
+            pixel_spacings = [
+                float(element.strip('"')) * 1000
+                for element in pixel_spacing_attribute.split(' ')
+            ]
+            mpp_x, mpp_y = pixel_spacings[0], pixel_spacings[1]
+            md[PROPERTY_NAME_MPP_X] = mpp_x
+            md[PROPERTY_NAME_MPP_Y] = mpp_y
+        return md
+
 
 def _parse_metadata_aperio(desc: str) -> dict[str, Any]:
     """Aperio SVS metadata"""

From 93f9e25e6e4a83d5804e449e9907e40cad3469e8 Mon Sep 17 00:00:00 2001
From: Erik O Gabrielsson <erik.o.gabrielsson@sectra.com>
Date: Wed, 29 Nov 2023 21:38:26 +0100
Subject: [PATCH 2/2] Remove unused import

---
 tiffslide/tiffslide.py | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tiffslide/tiffslide.py b/tiffslide/tiffslide.py
index 8d89fe2..7c241ca 100644
--- a/tiffslide/tiffslide.py
+++ b/tiffslide/tiffslide.py
@@ -14,7 +14,6 @@
 from typing import Literal
 from typing import Mapping
 from typing import Optional
-from typing import Tuple
 from typing import TypeVar
 from typing import overload
 from warnings import warn