use with context manager when reading/writing files

Author: Jaquier Aurélien Tristan

bluepyopt/deapext/algorithms.py

@@ -123,7 +123,8 @@ def eaAlphaMuPlusLambdaCheckpoint(
 
     if continue_cp:
         # A file name has been given, then load the data from the file
-        cp = pickle.load(open(cp_filename, "rb"))
+        with open(cp_filename, "rb") as f:
+            cp = pickle.load(f)
         population = cp["population"]
         parents = cp["parents"]
         start_gen = cp["generation"]
@@ -185,7 +186,8 @@ def eaAlphaMuPlusLambdaCheckpoint(
                       logbook=logbook,
                       rndstate=random.getstate(),
                       param_names=param_names)
-            pickle.dump(cp, open(cp_filename_tmp, "wb"))
+            with open(cp_filename_tmp, "wb") as f:
+                pickle.dump(cp, f)
             if os.path.isfile(cp_filename_tmp):
                 shutil.copy(cp_filename_tmp, cp_filename)
                 logger.debug('Wrote checkpoint to %s', cp_filename)

bluepyopt/deapext/optimisationsCMA.py

@@ -265,7 +265,8 @@ def run(
         if continue_cp:
 
             # A file name has been given, then load the data from the file
-            cp = pickle.load(open(cp_filename, "rb"))
+            with open(cp_filename, "rb") as f:
+                cp = pickle.load(f)
             gen = cp["generation"]
             self.hof = cp["halloffame"]
             logbook = cp["logbook"]
@@ -361,7 +362,8 @@ def run(
                     CMA_es=CMA_es,
                     param_names=param_names,
                 )
-                pickle.dump(cp, open(cp_filename_tmp, "wb"))
+                with open(cp_filename_tmp, "wb") as f:
+                    pickle.dump(cp, f)
                 if os.path.isfile(cp_filename_tmp):
                     shutil.copy(cp_filename_tmp, cp_filename)
                     logger.debug("Wrote checkpoint to %s", cp_filename)