Skip to content

Clean up GeoZarr output metadata: proj/spatial only, well-formed attributes #350

Open
0 of 7 issues completed

Description

@emmanuelmathot

Context

The independent-reader review in #349 shows duplicate and malformed metadata in the S2 L2A output of our GeoZarr engine. The Zarr Sample Service uses this engine in CPM, so these problems are in the published products.

Decisions

  1. proj, spatial and multiscales are the only normative georeferencing.
  2. The output makes no CF claim. We do not write a root Conventions attribute. CF is out of scope of the minispec (docs/geozarr-minispec.md). CF alignment is a spec topic (CF Convention Alignment zarr-developers/geozarr-spec#120).
  3. The GeoZarr V0 grid_mapping and spatial_ref are obsolete.
  4. CF-style attributes that we keep (flag_*, valid_*, scale_factor, add_offset) must be well-formed.
  5. _FillValue does not change. xarray needs it for Zarr v3 (fix: ensure fill_value for zarr v3 arrays round-trips pydata/xarray#11345).

Tasks

Downstream check

We searched titiler-eopf, titiler-openeo, data-pipeline, eodash-assets and the data-model tests. No code reads flag_meanings, short_name, the dtype attribute or Conventions. titiler-eopf copies proj:* and spatial:* from the group to the array (titiler/eopf/reader.py), so it does not need grid_mapping. We did not check eopf-stac.

Activity

  1. added
    enhancementNew feature or request
    cpm-driverGeoZarr driver for EOPF CPM (eopf convert-geozarr, engine geozarr)
    sample-serviceRoadmap with EODC for the EOPF Sentinel Zarr Samples Service (#277)
    on Nov 12, 2026
  2. d-v-b commented on Nov 12, 2026

    @d-v-b
    Contributor

    I'd like to see if #330 makes this fix easier to express than the current way of building the geozarr layout + attributes

  3. emmanuelmathot commented on Nov 12, 2026

    @emmanuelmathot
    ContributorAuthor

    I'd like to see if #330 makes this fix easier to express than the current way of building the geozarr layout + attributes

    Sure, please go ahead and if it solves them then we have a good argument to adopt it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    cpm-driverGeoZarr driver for EOPF CPM (eopf convert-geozarr, engine geozarr)enhancementNew feature or requestsample-serviceRoadmap with EODC for the EOPF Sentinel Zarr Samples Service (#277)

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions