From c0f153e55fd880fe3fab7fb1fca78856870f6a91 Mon Sep 17 00:00:00 2001 From: Jeroen Verstraelen Date: Mon, 22 Aug 2022 13:39:39 +0200 Subject: [PATCH] Issue #5 add merging for assets --- src/openeo_aggregator/backend.py | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/openeo_aggregator/backend.py b/src/openeo_aggregator/backend.py index 611b208c..028b77ef 100644 --- a/src/openeo_aggregator/backend.py +++ b/src/openeo_aggregator/backend.py @@ -191,12 +191,16 @@ def _merge_collection_metadata(self, by_backend: Dict[str, dict]) -> dict: cube_dimensions = getter.first("cube:dimensions") if cube_dimensions: result["cube:dimensions"] = cube_dimensions - # TODO merge existing summaries? - result["summaries"] = { - # TODO: use a more robust/user friendly backend pointer than backend id (which is internal implementation detail) - self.STAC_PROPERTY_PROVIDER_BACKEND: list(by_backend.keys()) - } - # TODO: assets ? + # summaries + # TODO: Better merging for summaries? + for summary in getter.get("summaries"): + print(summary) + result["summaries"] = {} + # TODO: use a more robust/user friendly backend pointer than backend id (which is internal implementation detail) + result["summaries"][self.STAC_PROPERTY_PROVIDER_BACKEND] = list(by_backend.keys()) + # assets + # TODO: assets ? For now, take the union. + result["assets"] = list(getter.union("assets")) return result