Recommend installing cftime when time decoding fails. (#4134)

dcherian · web-flow · commit e8bd8665e8fd · 2020-06-12T15:04:18.000Z
diff --git a/xarray/coding/times.py b/xarray/coding/times.py
@@ -80,8 +80,9 @@ def _decode_cf_datetime_dtype(data, units, calendar, use_cftime):
             "the default calendar" if calendar is None else "calendar %r" % calendar
         )
         msg = (
-            "unable to decode time units %r with %s. Try "
-            "opening your dataset with decode_times=False." % (units, calendar_msg)
+            f"unable to decode time units {units!r} with {calendar_msg!r}. Try "
+            "opening your dataset with decode_times=False or installing cftime "
+            "if it is not installed."
         )
         raise ValueError(msg)
     else:

Original file line number	Diff line number	Diff line change
`@@ -80,8 +80,9 @@ def _decode_cf_datetime_dtype(data, units, calendar, use_cftime):`
`80`	`80`	`"the default calendar" if calendar is None else "calendar %r" % calendar`
`81`	`81`	`)`
`82`	`82`	`msg = (`
`83`		`- "unable to decode time units %r with %s. Try "`
`84`		`- "opening your dataset with decode_times=False." % (units, calendar_msg)`
	`83`	`+ f"unable to decode time units {units!r} with {calendar_msg!r}. Try "`
	`84`	`+ "opening your dataset with decode_times=False or installing cftime "`
	`85`	`+ "if it is not installed."`
`85`	`86`	`)`
`86`	`87`	`raise ValueError(msg)`
`87`	`88`	`else:`