From b75c38526958a61064e086639d5391341e247e27 Mon Sep 17 00:00:00 2001 From: nehem Date: Fri, 9 Oct 2020 16:27:46 +1100 Subject: [PATCH] Ensure platform is landsat --- eodatasets3/model.py | 4 ++++ eodatasets3/properties.py | 2 +- 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/eodatasets3/model.py b/eodatasets3/model.py index ba0f9d2b..0cfaa6a9 100644 --- a/eodatasets3/model.py +++ b/eodatasets3/model.py @@ -438,6 +438,10 @@ def _dataset_label(self, sub_name: str = None): @property def platform_abbreviated(self) -> Optional[str]: # For now from Alchemist the platform is always landsat for C3 processing + if "ls" not in self.dataset.platform: + raise ValueError( + "ComplicatedNamingConventionsDerivatives assumes lansat as platform" + ) return "ls" diff --git a/eodatasets3/properties.py b/eodatasets3/properties.py index 32761966..68f53bd4 100644 --- a/eodatasets3/properties.py +++ b/eodatasets3/properties.py @@ -226,7 +226,7 @@ class StacPropertyView(collections.abc.Mapping): "landsat:wrs_path": int, "landsat:wrs_row": int, "odc:dataset_version": None, - "odc:collection_number": None, + "odc:collection_number": int, "odc:naming_conventions": None, # Not strict as there may be more added in ODC... "odc:file_format": of_enum_type(FileFormat, strict=False),