Insufficient cov details - timeout_cleanup by default

xscen/extract.py

@@ -1070,9 +1070,10 @@ def _subset_file_coverage(
             or guessed_nb_hrs_sum.nanos / period_nb_hrs.nanos < coverage
         ):
             logging.warning(
-                f"{df['id'].iloc[0] + ': ' if 'id' in df.columns else ''}Insufficient coverage "
-                f"({guessed_nb_hrs / period_nb_hrs}, {len(df[files_in_range])}, "
-                f"{guessed_nb_hrs_sum.nanos / period_nb_hrs.nanos}."
+                f"{df['id'].iloc[0] + ': ' if 'id' in df.columns else ''}Insufficient coverage."
+                f"% covered, min to max : {guessed_nb_hrs / period_nb_hrs:.1%}, "
+                f"% covered, sum of hours : {guessed_nb_hrs_sum.nanos / period_nb_hrs.nanos:.1%}, "
+                f"number of files in range : {len(df[files_in_range])}."
             )
             return pd.DataFrame(columns=df.columns)
 

xscen/io.py

@@ -13,7 +13,7 @@
 from xclim.core.calendar import get_calendar
 
 from .config import parse_config
-from .scripting import TimeoutException, timeout
+from .scripting import TimeoutException
 from .utils import translate_time_chunk
 
 logger = logging.getLogger(__name__)
@@ -359,7 +359,7 @@ def save_to_zarr(
     encoding: dict = None,
     mode: str = "f",
     itervar: bool = False,
-    timeout_cleanup: bool = False,
+    timeout_cleanup: bool = True,
 ) -> None:
     """
     Saves a Dataset to Zarr, rechunking if requested.
@@ -388,7 +388,7 @@ def save_to_zarr(
       If True, (data) variables are written one at a time, appending to the zarr.
       If False, this function computes, no matter what was passed to kwargs.
     timeout_cleanup : bool
-      If True and a :py:class:`xscen.scripting.TimeoutException` is raised during the writing,
+      If True (default) and a :py:class:`xscen.scripting.TimeoutException` is raised during the writing,
       the variable being written is removed from the dataset as it is incomplete.
       This does nothing if `compute` is False.