Pretty `_repr_html_` of cube dimension metadata by soxofaan · Pull Request #955 · Open-EO/openeo-python-client

soxofaan · 2026-10-09T13:00:05Z

soxofaan · 2026-10-09T15:51:52Z

Current state of the visualization (just table for now):

note that the temporal and spatial extents are still the unfiltered extents of the original collection, instead of the actual cube extents

Copilot

🟡 Changes recommended

Temporal extent handling and HTML rendering contain runtime failures for supported metadata and open-ended intervals.

7 open findings

Allow metadata without a temporal dimension · New
Coerce dimension types before HTML escaping · New
Import dateutil.parser explicitly · New
Add tests for interval conversion and intersection · New
Allow None in temporal extent annotations · New
Define stable representation for non-concrete extents · New
Add VectorCube HTML representation tests · New

What changed in this PR

Adds richer Jupyter rendering for cube metadata, addressing #827, while tracking temporal extents through filtering operations.

Changes:

Adds HTML dimension tables to raster and vector cube representations.
Introduces temporal interval intersection and extent markers.
Adds rendering tests and a changelog entry.

File	Description
`CHANGELOG.md`	Documents the Jupyter representation.
`openeo/dates.py`	Adds datetime migration note.
`openeo/metadata/__init__.py`	Adds rendering and temporal filtering.
`openeo/metadata/_extents.py`	Defines special extent markers.
`openeo/metadata/_jupyter.py`	Renders dimension metadata tables.
`openeo/rest/_datacube.py`	Names process-graph HTML output.
`openeo/rest/datacube.py`	Combines metadata and graph rendering.
`openeo/rest/vectorcube.py`	Adds metadata rendering to vector cubes.
`openeo/util.py`	Adds datetime migration notes.
`openeo/utils/datetime.py`	Implements datetime intervals.
`tests/metadata/test_jupyter.py`	Tests table rendering.
`tests/metadata/test_metadata.py`	Tests metadata HTML output.
`tests/rest/datacube/test_datacube.py`	Tests cube HTML composition.

🧠 Review effort: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Copilot

🟡 Changes recommended

Open-ended temporal intervals can raise errors, and the public TemporalDimension constructor is unintentionally breaking.

3 open findings

6 resolved since last review

🧠 Review effort: Balanced

Copilot

🟡 Changes recommended

Half-open temporal filters can currently produce a non-empty metadata extent when the query contains no matching instants.

5 open findings

2 resolved since last review

🧠 Review effort: Balanced

Copilot

🟡 Changes recommended

Dynamic temporal bounds can raise errors, and successive filters lose exclusive-end semantics.

1 open finding

Dynamic temporal bounds are incorrectly parsed as concrete dates · New

5 resolved since last review

🧠 Review effort: Balanced

…955

…nterval #955

Copilot

🟡 Changes recommended

Successive temporal filters lose end-exclusive semantics and can produce an incorrect non-empty extent.

1 open finding

Preserve end-exclusive semantics when serializing intersections · New

1 resolved since last review

Dynamic temporal bounds are incorrectly parsed as concrete dates

🧠 Review effort: Balanced

…955

Copilot

🟢 Approval recommended

The rendering and temporal metadata behavior are consistently implemented and covered by focused tests.

0 open findings

1 resolved since last review

Preserve end-exclusive semantics when serializing intersections

🧠 Review effort: Balanced

related to #955, #827

(re)group test functions in relevant test classes for better overview and easier future evolution refs #949 #827 #955

Copilot

🟡 Changes recommended

Supported unnamed STAC bands cause the new notebook renderer to raise TypeError.

1 open finding

Handle unnamed STAC bands in Jupyter display · New

🧠 Review effort: Balanced

Trying to keep all this into account: - keep existing public API - support for parameterized/dynamic `filter_temporal` - handle empty temporal extent properly - calculate real temporal intersection - don't require metadata refs #827, #949

related to #955, #827

(re)group test functions in relevant test classes for better overview and easier future evolution refs #949 #827 #955

refs #827, #949

…Dimension

…SpatialDimension

…Dimension

Copilot

🟡 Changes recommended

Spatial metadata propagation has correctness regressions, and existing REST test expectations remain incompatible with the new representation.

5 open findings

1 resolved since last review

Handle unnamed STAC bands in Jupyter display

🧠 Review effort: Balanced

+    def spatial_dimensions(self) -> SpatialDimensionsXY:
+        if not self.has_spatial_dimensions():
+            raise MetadataException("No spatial dimensions")
+        return self._spatial_dimensions


+        if isinstance(extent, dict) and {"west", "east", "south", "north"}.issubset(extent):
+            bbox = BBoxDict.from_dict(extent)


+            bbox_crs = bbox.get("crs", SpatialDimension.DEFAULT_CRS)
+            if bbox_crs == self.x.crs and bbox_crs == self.y.crs:
+                # No reprojection needed
+                extent_x = bbox["west"], bbox["east"]


+
+    def filter_bbox(self, extent: Any) -> CubeMetadata:
+        if self._dimensions and isinstance(self._spatial_dimensions, SpatialDimensionsXY):
+            spatial_dimensions: SpatialDimensionsXY = self._spatial_dimensions.filter_bbox(extent=extent)


+        if metadata and spatial_extent:
+            metadata = metadata.filter_bbox(extent=spatial_extent)


…SpatialDimension

…Dimension

soxofaan force-pushed the issue827-cube-metadata-viz branch from c24bb72 to fbbf322 Compare October 9, 2026 13:01

soxofaan changed the title ~~Rudimentary _repr_html_ of cube dimension metadata~~ Pretty _repr_html_ of cube dimension metadata Oct 9, 2026

soxofaan requested a balanced review from Copilot October 9, 2026 21:47

Copilot started reviewing on behalf of soxofaan October 9, 2026 21:47

View session

Copilot AI reviewed Oct 9, 2026

View reviewed changes

soxofaan force-pushed the issue827-cube-metadata-viz branch from 54f4316 to 9b9f13f Compare October 12, 2026 10:19

soxofaan requested a balanced review from Copilot October 12, 2026 10:20

Copilot started reviewing on behalf of soxofaan October 12, 2026 10:20

View session

Copilot AI reviewed Oct 12, 2026

View reviewed changes

Comment thread openeo/utils/datetime.py Outdated

Comment thread openeo/metadata/__init__.py

soxofaan force-pushed the issue827-cube-metadata-viz branch 2 times, most recently from 281ce2b to a2ab17a Compare October 12, 2026 13:16

soxofaan requested a balanced review from Copilot October 12, 2026 13:16

Copilot started reviewing on behalf of soxofaan October 12, 2026 13:17

View session

Copilot AI reviewed Oct 12, 2026

View reviewed changes

Comment thread openeo/metadata/__init__.py Outdated

Comment thread openeo/metadata/__init__.py Outdated

Comment thread openeo/utils/datetime.py

Comment thread openeo/utils/datetime.py Outdated

soxofaan added a commit that referenced this pull request Oct 12, 2026

Handle filter_temporal end-exclusiveness in DateTimeInterval #955

811d9fa

soxofaan requested a balanced review from Copilot October 12, 2026 16:37

Copilot started reviewing on behalf of soxofaan October 12, 2026 16:38

View session

Copilot AI reviewed Oct 12, 2026

View reviewed changes

Comment thread openeo/metadata/__init__.py Outdated

soxofaan added a commit that referenced this pull request Oct 12, 2026

fixup! Handle filter_temporal end-exclusiveness in DateTimeInterval #…

7eff22a

…955

soxofaan added a commit that referenced this pull request Oct 12, 2026

fixup! Handle filter_temporal end-exclusiveness in DateTimeInterval #…

cc94f24

…955

soxofaan added a commit that referenced this pull request Oct 12, 2026

fixup! fixup! Handle filter_temporal end-exclusiveness in DateTimeI…

3934dab

…nterval #955

soxofaan added a commit that referenced this pull request Oct 12, 2026

Handle filter_temporal end-exclusiveness in DateTimeInterval #955

e288323

soxofaan force-pushed the issue827-cube-metadata-viz branch from 3934dab to e288323 Compare October 12, 2026 19:12

soxofaan requested a balanced review from Copilot October 12, 2026 19:17

Copilot started reviewing on behalf of soxofaan October 12, 2026 19:18

View session

Copilot AI reviewed Oct 12, 2026

View reviewed changes

Comment thread openeo/metadata/__init__.py

soxofaan added a commit that referenced this pull request Oct 13, 2026

fixup! Handle filter_temporal end-exclusiveness in DateTimeInterval #…

48dac2c

…955

soxofaan requested a balanced review from Copilot October 13, 2026 07:01

Copilot started reviewing on behalf of soxofaan October 13, 2026 07:02

View session

Copilot AI reviewed Oct 13, 2026

View reviewed changes

soxofaan added a commit that referenced this pull request Oct 13, 2026

Handle filter_temporal end-exclusiveness in DateTimeInterval #955

5d6157c

soxofaan force-pushed the issue827-cube-metadata-viz branch from 48dac2c to 5d6157c Compare October 13, 2026 07:40

soxofaan self-assigned this Oct 13, 2026

soxofaan added a commit that referenced this pull request Oct 15, 2026

Add axis attribute to SpatialDimension

b657b8e

related to #955, #827

soxofaan added a commit that referenced this pull request Oct 15, 2026

Reorganise test_metadata.py

ffcdd97

(re)group test functions in relevant test classes for better overview and easier future evolution refs #949 #827 #955

soxofaan marked this pull request as draft October 15, 2026 13:53

soxofaan requested a balanced review from Copilot October 16, 2026 12:22

Copilot started reviewing on behalf of soxofaan October 16, 2026 12:23

View session

Copilot AI reviewed Oct 16, 2026

View reviewed changes

Comment thread openeo/metadata/_jupyter.py Outdated

soxofaan added 6 commits October 16, 2026 17:40

Initial table-based repr_html of cube dimension metadata #827

ead4ed7

Handle filter_temporal end-exclusiveness in DateTimeInterval #955

e28a735

Add axis attribute to SpatialDimension

deccd7a

related to #955, #827

Reorganise test_metadata.py

e2cc888

(re)group test functions in relevant test classes for better overview and easier future evolution refs #949 #827 #955

Basic handling of filter_bbox in CubeMetadata, down to SpatialDimension

1cb1141

refs #827, #949

soxofaan force-pushed the issue827-cube-metadata-viz branch from 2c9608e to 1cb1141 Compare October 16, 2026 15:42

soxofaan added 2 commits October 16, 2026 17:48

fixup! Basic handling of filter_bbox in CubeMetadata, down to Spatial…

eea63d5

…Dimension

fixup! fixup! Basic handling of filter_bbox in CubeMetadata, down to …

f4007b5

…SpatialDimension

soxofaan requested a balanced review from Copilot October 16, 2026 15:49

Copilot started reviewing on behalf of soxofaan October 16, 2026 15:50

View session

fixup! Basic handling of filter_bbox in CubeMetadata, down to Spatial…

43bc219

…Dimension

Copilot AI reviewed Oct 16, 2026

View reviewed changes

soxofaan added 2 commits October 16, 2026 18:10

fixup! fixup! Basic handling of filter_bbox in CubeMetadata, down to …

4f07a54

…SpatialDimension

fixup! Basic handling of filter_bbox in CubeMetadata, down to Spatial…

ae611a3

…Dimension

		if isinstance(extent, dict) and {"west", "east", "south", "north"}.issubset(extent):
		bbox = BBoxDict.from_dict(extent)

		if metadata and spatial_extent:
		metadata = metadata.filter_bbox(extent=spatial_extent)

Conversation

soxofaan commented Oct 9, 2026

Uh oh!

soxofaan commented Oct 9, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟢 Approval recommended

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

This branch has not been deployed

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants