Fix compatibility with xarray 0.14.1 (#69)

benbovy · web-flow · commit a3fd84aebebe · 2019-12-10T10:30:20.000+01:00
* use drop_vars instead of drop

* revert to using Dataset.drop

Dataset.drop_vars has been added in xarray 0.14.1 and we certainly want
to support older versions. The annoyance is minor (tests warnings).
diff --git a/xsimlab/drivers.py b/xsimlab/drivers.py
@@ -277,7 +277,7 @@ def _get_runtime_datasets(self):
             '_clock_diff': mclock_coord.diff(mclock_dim, label='lower')
         }
 
-        ds_all_steps = (self.dataset.drop(ds_init.data_vars.keys(),
+        ds_all_steps = (self.dataset.drop(list(ds_init.data_vars.keys()),
                                           errors='ignore')
                                     .isel({mclock_dim: slice(0, -1)})
                                     .assign(step_data_vars))

Original file line number	Diff line number	Diff line change
`@@ -277,7 +277,7 @@ def _get_runtime_datasets(self):`
`277`	`277`	`'_clock_diff': mclock_coord.diff(mclock_dim, label='lower')`
`278`	`278`	`}`
`279`	`279`
`280`		`- ds_all_steps = (self.dataset.drop(ds_init.data_vars.keys(),`
	`280`	`+ ds_all_steps = (self.dataset.drop(list(ds_init.data_vars.keys()),`
`281`	`281`	`errors='ignore')`
`282`	`282`	`.isel({mclock_dim: slice(0, -1)})`
`283`	`283`	`.assign(step_data_vars))`