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

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

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

ref #827 #955

refs #827, #949

ref #827 #955

Copilot

🟡 Changes recommended

Resampling with incomplete target metadata can now raise an exception and prevent process graph construction.

1 open issue

Avoid resampling failure for targets with partial spatial metadata · New

6 resolved since last review

_{Review effort: Balanced}

Copilot

🟡 Changes recommended

Cubes with a z axis lose horizontal-dimension detection and can lose spatial metadata during resampling.

1 open issue

Handle x/y spatial dimensions when additional axes are present · New

1 resolved since last review

Avoid resampling failure for targets with partial spatial metadata

_{Review effort: Balanced}

+            if spatial_dims := sorted((d for d in dimensions if isinstance(d, SpatialDimension)), key=lambda d: d.axis):
+                if [d.axis for d in spatial_dims] == ["x", "y"]:
+                    self._spatial_dimensions = SpatialDimensionsXY(x=spatial_dims[0], y=spatial_dims[1])


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

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 5 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

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

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

Copilot AI reviewed Oct 16, 2026

View reviewed changes

Comment thread openeo/metadata/__init__.py

Comment thread openeo/metadata/__init__.py Outdated

Comment thread openeo/metadata/__init__.py

Comment thread openeo/metadata/__init__.py

Comment thread openeo/rest/datacube.py

soxofaan force-pushed the issue827-cube-metadata-viz branch from fc3d941 to fc76003 Compare October 19, 2026 10:15

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

Track SpatialDimension.crs from _StacMetadataParser

48485bc

ref #827 #955

soxofaan added 4 commits October 20, 2026 16:07

Basic handling of filter_bbox in CubeMetadata, down to SpatialDimension

6e2c808

refs #827, #949

Exclude antimeridian handling from metadata filter_bbox for now

17d48f2

refs #827, #949

minor #867 fixup

9545d61

Track SpatialDimension.crs from _StacMetadataParser

868cdc8

ref #827 #955

soxofaan force-pushed the issue827-cube-metadata-viz branch from b37c6bb to 868cdc8 Compare October 20, 2026 14:07

soxofaan requested a balanced review from Copilot October 20, 2026 14:08

Copilot started reviewing on behalf of soxofaan October 20, 2026 14:09

View session

Copilot AI reviewed Oct 20, 2026

View reviewed changes

Comment thread openeo/metadata/__init__.py

fixup! Track SpatialDimension.crs from _StacMetadataParser

0a956cc

soxofaan requested a balanced review from Copilot October 20, 2026 14:23

Copilot started reviewing on behalf of soxofaan October 20, 2026 14:23

View session

Copilot AI reviewed Oct 20, 2026

View reviewed changes

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

🟡 Changes recommended

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!

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