Merge 0.13.8 into develop (#674)

* Do not modifiy xarray engine kwargs dicts in place (#671)

* Add release notes for 0.13.8 (#673)

Author: sandorkertesz

docs/release_notes/version_0.13_updates.rst

@@ -2,6 +2,15 @@ Version 0.13 Updates
 /////////////////////////
 
 
+Version 0.13.8
+===============
+
+Fixes
+++++++++++
+
+- Fixed issue when kwargs passed to the Xarray engine as a dict were modified in-place (:pr:`671`).
+
+
 Version 0.13.7
 ===============
 

src/earthkit/data/readers/grib/xarray.py

@@ -7,6 +7,7 @@
 # nor does it submit to any jurisdiction.
 #
 
+import copy
 import logging
 
 from earthkit.data.utils import ensure_dict
@@ -344,6 +345,7 @@ def to_xarray(self, engine="earthkit", xarray_open_dataset_kwargs=None, **kwargs
             raise ValueError(f"Unsupported engine: {engine}. Please use one of {list(engines.keys())}")
 
         user_xarray_open_dataset_kwargs = ensure_dict(xarray_open_dataset_kwargs)
+        user_xarray_open_dataset_kwargs = copy.deepcopy(user_xarray_open_dataset_kwargs)
         if user_xarray_open_dataset_kwargs and kwargs:
             raise ValueError("Cannot specify extra keyword arguments when xarray_open_dataset_kwargs is set.")
 

tests/xr_engine/test_xr_engine.py

@@ -23,6 +23,17 @@
 from xr_engine_fixtures import load_grib_data  # noqa: E402
 
 
+@pytest.mark.cache
+@pytest.mark.parametrize("engine", ["earthkit", "cfgrib"])
+def test_xr_engine_kwargs_unchanged(engine):
+    ds = from_source("url", earthkit_remote_test_data_file("test-data/xr_engine/level/pl_small.grib"))
+
+    _kwargs = {"squeeze": True}
+    res = ds.to_xarray(engine=engine, xarray_open_dataset_kwargs=_kwargs)
+    assert res is not None
+    assert _kwargs == {"squeeze": True}
+
+
 @pytest.mark.cache
 @pytest.mark.parametrize(
     "file",