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

🟡 Changes recommended

Parameterized resampling replaces an unknown coordinate system with EPSG:4326, allowing incorrect spatial metadata filtering.

1 open issue

Preserve unknown CRS for parameterized projections · New

_{Review effort: Balanced}

ref #827 #955

Copilot

🟡 Changes recommended

Three resampling test assertions still expect values incompatible with the new unknown-value sentinel.

3 open issues

1 resolved since last review

Preserve unknown CRS for parameterized projections

_{Review effort: Balanced}

ref #827 #955

Copilot

🔵 Needs a closer look

Public metadata API changes need maintainer compatibility review, and an unresolved metadata correctness issue remains.

1 open issue

Normalize CRS values before comparing bbox and dimension references · New

3 resolved since last review

_{Review effort: Balanced}

Normalize the dimension CRS (e.g. "EPSG:4326") before comparing it with the bbox CRS, so equivalent CRS notations no longer fall back to a non-concrete extent. Add `normalize_crs_or_passthrough` helper that leaves unparseable CRS values untouched instead of raising. ref #827 #955

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

ref #827 #955

…cubes with z dimension' Co-authored-by: Copilot Autofix powered by AI <[email protected]>

ref #827 #955

Normalize the dimension CRS (e.g. "EPSG:4326") before comparing it with the bbox CRS, so equivalent CRS notations no longer fall back to a non-concrete extent. Add `normalize_crs_or_passthrough` helper that leaves unparseable CRS values untouched instead of raising. ref #827 #955

Copilot

🔵 Needs a closer look

Public metadata API changes and cross-cutting extent propagation need final maintainer validation beyond the notebook-rendering feature.

No open issues 🎉

1 resolved since last review

Normalize CRS values before comparing bbox and dimension references

_{Review effort: Balanced}

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 started reviewing on behalf of soxofaan October 20, 2026 16:10

View session

Copilot AI reviewed Oct 20, 2026

View reviewed changes

Comment thread openeo/metadata/__init__.py

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

Improve default crs handling in SpatialDimension

ebee466

ref #827 #955

soxofaan force-pushed the issue827-cube-metadata-viz branch from 140eba7 to d591b11 Compare October 22, 2026 09:27

soxofaan requested a balanced review from Copilot October 22, 2026 09:27

Copilot started reviewing on behalf of soxofaan October 22, 2026 09:28

View session

Copilot AI reviewed Oct 22, 2026

View reviewed changes

Comment thread tests/rest/datacube/test_datacube.py Outdated

Comment thread tests/rest/datacube/test_datacube.py Outdated

Comment thread tests/rest/datacube/test_datacube.py Outdated

soxofaan force-pushed the issue827-cube-metadata-viz branch from 682af00 to 6ebb220 Compare October 22, 2026 09:40

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

Track SpatialDimension.crs from _StacMetadataParser

aaea826

ref #827 #955

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

Ignore "z" spatial dimension in CubeMetadata

de3cdca

ref #827 #955

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

Improve default crs handling in SpatialDimension

6ebb220

ref #827 #955

soxofaan requested a balanced review from Copilot October 22, 2026 09:41

Copilot started reviewing on behalf of soxofaan October 22, 2026 09:42

View session

Copilot AI reviewed Oct 22, 2026

View reviewed changes

Comment thread openeo/metadata/__init__.py Outdated

soxofaan and others added 13 commits October 23, 2026 17:50

Initial table-based repr_html of cube dimension metadata #827

5104897

Handle filter_temporal end-exclusiveness in DateTimeInterval #955

122be58

Add axis attribute to SpatialDimension

f21697f

related to #955, #827

Reorganise test_metadata.py

2b1e7d9

(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

ba9d807

refs #827, #949

Exclude antimeridian handling from metadata filter_bbox for now

56b13b9

refs #827, #949

minor #867 fixup

0dc9265

Track SpatialDimension.crs from _StacMetadataParser

d7ab05e

ref #827 #955

Ignore "z" spatial dimension in CubeMetadata

7d89925

ref #827 #955

Potential fix for pull request finding 'filter_bbox crashes on x/y/z …

2cbabca

…cubes with z dimension' Co-authored-by: Copilot Autofix powered by AI <[email protected]>

Improve default crs handling in SpatialDimension

502fc50

ref #827 #955

soxofaan force-pushed the issue827-cube-metadata-viz branch from fac20d5 to 035e70a Compare October 23, 2026 15:51

Copilot AI reviewed Oct 23, 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!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

This branch has not been deployed

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants