BREAKING CHANGE**: Using now dict2xml rather than dicttoxml #292

Author: remi-braun

CHANGES.md

@@ -2,6 +2,7 @@
 
 ## 0.24.0 (2026-mm-dd)
 
+- **BREAKING CHANGE**: Using now `dict2xml` rather than `dicttoxml` [#292](https://github.com/sertit/eoreader/issues/292)
 - **ENH: Adding the support of `Satellogic` (`Aleph-1`) constellation** [#229](https://github.com/sertit/eoreader/issues/229)
 - **ENH: Add fallback for TSX/TDX extent when `SUPPORT/GEARTH_POLY.kml` is missing** [#290](https://github.com/sertit/eoreader/issues/290)
 - **ENH: Handle windows when loading DEM bands**

eoreader/products/optical/aleph1_product.py

@@ -24,7 +24,6 @@
 import geopandas as gpd
 import numpy as np
 import xarray as xr
-from dicttoxml import dicttoxml
 from lxml import etree
 from rasterio.enums import Resampling
 from sertit import files, path
@@ -525,6 +524,8 @@ def _read_mtd(self) -> (etree._Element, dict):
         """
         # MTD are JSON
         try:
+            from dict2xml import dict2xml
+
             mtd = self._read_mtd_dict()
 
             # Sanitize STAC mtd (remove STAC prefixes with ':' that break XML keys)
@@ -541,7 +542,7 @@ def __sanitize_recursive(d):
             mtd.pop("stac_version", None)
             mtd.pop("type", None)
             __sanitize_recursive(mtd)
-            root = etree.fromstring(dicttoxml(mtd, attr_type=False))
+            root = etree.fromstring(dict2xml(mtd, wrap="all"))
         except etree.XMLSyntaxError as exc:
             raise InvalidProductError(
                 f"Cannot convert metadata to XML for {self.path}!"

eoreader/products/sar/capella_product.py

@@ -25,7 +25,6 @@
 
 import geopandas as gpd
 from affine import Affine
-from dicttoxml import dicttoxml
 from lxml import etree
 from rasterio import CRS, transform
 from sertit import files, path, vectors
@@ -409,6 +408,8 @@ def _read_mtd(self) -> (etree._Element, dict):
 
         # MTD are JSON
         try:
+            from dict2xml import dict2xml
+
             try:
                 mtd_file = next(self.path.glob(f"{self.name}.json"))
                 self._has_stac_mtd = True
@@ -425,7 +426,17 @@ def _read_mtd(self) -> (etree._Element, dict):
                     ) from ex
 
             data = files.read_json(mtd_file, print_file=False)
-            root = etree.fromstring(dicttoxml(data))
+
+            def __sanitize_recursive(d):
+                for key in d.copy():
+                    k = key.split(":")[-1]
+                    d[k] = d.pop(key)
+                    if isinstance(d[k], dict):
+                        __sanitize_recursive(d[k])
+
+            data.pop("assets", None)
+            __sanitize_recursive(data)
+            root = etree.fromstring(dict2xml(data, wrap="all"))
         except etree.XMLSyntaxError as exc:
             raise InvalidProductError(
                 f"Cannot convert metadata to XML for {self.path}!"
@@ -493,7 +504,7 @@ def get_orbit_direction(self) -> OrbitDirection:
         if self._has_stac_mtd:
             root, _ = self.read_mtd()
 
-            ob = root.findtext(".//key[@name='sat:orbit_state']")
+            ob = root.findtext(".//orbit_state")
             ob = OrbitDirection.from_value(ob.upper())
 
         if ob is None:

eoreader/products/sar/umbra_product.py

@@ -26,7 +26,6 @@
 import geopandas as gpd
 import numpy as np
 import rasterio
-from dicttoxml import dicttoxml
 from lxml import etree
 from rasterio import crs
 from sertit import files, path, vectors
@@ -428,9 +427,10 @@ def _read_mtd(self) -> (etree._Element, dict):
         Returns:
             (etree._Element, dict): Metadata XML root and its namespaces as a dict
         """
-
         # MTD are JSON
         try:
+            from dict2xml import dict2xml
+
             try:
                 mtd_file = self._get_stac_mtd_path()
                 self._has_stac_mtd = True
@@ -462,7 +462,7 @@ def __sanitize_recursive(d):
 
                 data.pop("assets", None)
                 __sanitize_recursive(data)
-            root = etree.fromstring(dicttoxml(data, attr_type=False))
+            root = etree.fromstring(dict2xml(data, wrap="all"))
         except etree.XMLSyntaxError as exc:
             raise InvalidProductError(
                 f"Cannot convert metadata to XML for {self.path}!"
@@ -488,7 +488,7 @@ def get_raw_band_paths(self, **kwargs) -> dict:
 
         # Open polarizations
         # Same for the two MTD
-        pol = root.find(".//polarizations").findtext("item")
+        pol = root.findtext(".//polarizations")
 
         # Convert pol to an EOReader band
         pol = sab.from_value(pol)

pyproject.toml

@@ -51,7 +51,7 @@ dependencies = [
     "cloudpathlib[s3]>=0.15.0",
     "validators",
     "methodtools",
-    "dicttoxml",
+    "dict2xml",
     "tifffile",
     "ephem",
 ]

requirements.txt

@@ -12,7 +12,7 @@ twine
 lxml
 zarr
 h5netcdf
-dicttoxml
+dict2xml
 
 # Paths
 validators