From 7c982c66630cda7e02a3ea2dd56b01dd3346f9bb Mon Sep 17 00:00:00 2001 From: Raphael Hagen Date: Thu, 19 Oct 2023 12:52:04 -0700 Subject: [PATCH 1/2] added check to ensure consolidated=True is not passed to open_dataset --- kerchunk/xarray_backend.py | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/kerchunk/xarray_backend.py b/kerchunk/xarray_backend.py index badf2907..32480501 100644 --- a/kerchunk/xarray_backend.py +++ b/kerchunk/xarray_backend.py @@ -48,6 +48,12 @@ def open_reference_dataset( if open_dataset_options is None: open_dataset_options = {} + if open_dataset_options.get("consolidated"): + raise ValueError( + "consolidated = True is not supported when opening Kerchunk reference files through Xarray." + ) + open_dataset_options["consolidated"] = False + m = fsspec.get_mapper("reference://", fo=filename_or_obj, **storage_options) return xr.open_dataset(m, engine="zarr", consolidated=False, **open_dataset_options) From a60d5f7ff8311472d9dd503d1b2068ac67b9944a Mon Sep 17 00:00:00 2001 From: Raphael Hagen Date: Thu, 19 Oct 2023 13:00:30 -0700 Subject: [PATCH 2/2] removed duplicate --- kerchunk/xarray_backend.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kerchunk/xarray_backend.py b/kerchunk/xarray_backend.py index 32480501..ded510bb 100644 --- a/kerchunk/xarray_backend.py +++ b/kerchunk/xarray_backend.py @@ -56,4 +56,4 @@ def open_reference_dataset( m = fsspec.get_mapper("reference://", fo=filename_or_obj, **storage_options) - return xr.open_dataset(m, engine="zarr", consolidated=False, **open_dataset_options) + return xr.open_dataset(m, engine="zarr", **open_dataset_options)