BUG: parse ODF time values with comments

Author: mzuzek

doc/source/whatsnew/v2.2.0.rst

@@ -252,6 +252,7 @@ Bug fixes
 - Bug in :class:`pandas.core.window.Rolling` where duplicate datetimelike indexes are treated as consecutive rather than equal with ``closed='left'`` and ``closed='neither'`` (:issue:`20712`)
 - Bug in :meth:`DataFrame.apply` where passing ``raw=True`` ignored ``args`` passed to the applied function (:issue:`55009`)
 - Bug in :meth:`pandas.read_excel` with a ODS file without cached formatted cell for float values (:issue:`55219`)
+- Bug in :meth:`pandas.read_excel` where ODS files with comments on time value cells failed to parse (related to :issue:`55200`)
 
 Categorical
 ^^^^^^^^^^^

pandas/io/excel/_odfreader.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import re
 from typing import (
     TYPE_CHECKING,
     cast,
@@ -182,6 +183,20 @@ def _get_column_repeat(self, cell) -> int:
 
         return int(cell.attributes.get((TABLENS, "number-columns-repeated"), 1))
 
+    def _parse_odf_time(self, value: str) -> pd.Timestamp:
+        """
+        Helper function to convert ODF variant of ISO 8601 formatted duration
+        "PnYnMnDTnHnMnS" - see https://www.w3.org/TR/xmlschema-2/#duration
+        """
+        parts = re.match(r"^\s*PT\s*(\d+)\s*H\s*(\d+)\s*M\s*(\d+(\.\d+)?)\s*S$", value)
+        if parts is None:
+            raise ValueError(f"Failed to parse ODF time value: {value}")
+        h, m, s = parts.group(1, 2, 3)
+        # ignore date part from some representations as both pd.Timestamp
+        # and datetime.time restrict hour values to 0..23
+        h = str(int(h) % 24)
+        return pd.Timestamp(f"{h}:{m}:{s}")
+
     def _get_cell_value(self, cell) -> Scalar | NaTType:
         from odf.namespaces import OFFICENS
 
@@ -214,7 +229,8 @@ def _get_cell_value(self, cell) -> Scalar | NaTType:
             cell_value = cell.attributes.get((OFFICENS, "date-value"))
             return pd.Timestamp(cell_value)
         elif cell_type == "time":
-            stamp = pd.Timestamp(str(cell))
+            cell_value = cell.attributes.get((OFFICENS, "time-value"))
+            stamp = self._parse_odf_time(str(cell_value))
             # cast needed here because Scalar doesn't include datetime.time
             return cast(Scalar, stamp.time())
         else: