Skip to content

Sentinel-3 RBT processing omits consolidation for measurement group #315

Open

Description

@shollendEODC

If the generic_rechunker functionality is applied to Sentinel-3 SLSTR RBT the measurement group is not consolidated as its skipped during the rechunking and not added in later processes. This is theoretically not an issue as S3 products are currently routed throug

generic_converter.py lines 62-72 in 7f6a22616da36a442501ac27b63fb001e0470c65

# Skip parent groups that have children (only process leaf groups)
        if (
            hasattr(group_node, "children")
            and len(group_node.children) > 0
            and len(group_node.data_vars) == 0
        ):
            # this catches empty groups which have children but no data variables themselves. This allows the (theoretical)
            # application of the gener_rechunker to S3 OLC data products as they tend to have data variables on
            # /measurements or other groups, while also having an orphan as a child.
            # This extended check does not limit other products, as verified by the test suite.
            continue

The solution can be integrated by adding skipped groups to processed_groups with None, which triggers its inclusion and thus consolidation during the updated_root_consolidation(...) call.

Activity

  1. self-assigned this
    on Oct 14, 2026
  2. added
    bugSomething isn't working
    sentinel-3Sentinel-3 products (OLCI, SLSTR)
    on Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't workingsentinel-3Sentinel-3 products (OLCI, SLSTR)

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions