Merge pull request #604 from benjwadams/dealloc_ds_error_fix

Dealloc ds error fix

compliance_checker/__init__.py

@@ -10,6 +10,12 @@
     from functools32 import lru_cache
 
 class MemoizedDataset(Dataset):
+    """
+    A NetCDF dataset which has its get_variables_by_attributes call memoized in
+    order to speed up repeated calls to the function.  This should only really
+    be used against netCDF Datasets opened in 'r' mode, as the attributes should
+    not change upon reading the files.
+    """
     @lru_cache(128)
     def get_variables_by_attributes(self, **kwargs):
         return super(MemoizedDataset,

compliance_checker/runner.py

@@ -66,6 +66,9 @@ def run_checker(cls, ds_loc, checker_names, verbose, criteria,
 
             score_groups = cs.run(ds, [] if skip_checks is None else skip_checks,
                                 *checker_names)
+            # TODO: consider wrapping in a proper context manager instead
+            if hasattr(ds, 'close'):
+                ds.close()
 
             if not score_groups:
                 raise ValueError("No checks found, please check the name of the checker(s) and that they are installed")