no need to explicitly cleanup

Author: aaronwmorris

indi_allsky/darks.py

@@ -829,7 +829,7 @@ def _run(self, stacking_class):
     def _take_exposures(self, exposure, dark_filename_t, bpm_filename_t, ccd_bits, stacking_class):
         exposure_f = float(exposure)
 
-        tmp_fit_dir = tempfile.TemporaryDirectory()
+        tmp_fit_dir = tempfile.TemporaryDirectory()    # context manager automatically deletes files when finished
         tmp_fit_dir_p = Path(tmp_fit_dir.name)
 
         logger.info('Temp folder: %s', tmp_fit_dir_p)

indi_allsky/starTrails.py

@@ -72,13 +72,10 @@ def __init__(self, config, bin_v, mask=None):
             self.image_dir = Path(__file__).parent.parent.joinpath('html', 'images').absolute()
 
 
-        self.timelapse_tmpdir = tempfile.TemporaryDirectory(dir=self.image_dir, suffix='_startrail_timelapse')
+        self.timelapse_tmpdir = tempfile.TemporaryDirectory(dir=self.image_dir, suffix='_startrail_timelapse')    # context manager automatically deletes files when finished
         self.timelapse_tmpdir_p = Path(self.timelapse_tmpdir.name)
 
 
-    def __del__(self):
-        self.cleanup()
-
 
     @property
     def max_brightness(self):

indi_allsky/timelapse.py

@@ -17,12 +17,11 @@ class TimelapseGenerator(object):
     def __init__(self, config):
         self.config = config
 
-        self.seqfolder = tempfile.TemporaryDirectory(suffix='_timelapse')
+        self.seqfolder = tempfile.TemporaryDirectory(suffix='_timelapse')  # context manager automatically deletes files when finished
         self.seqfolder_p = Path(self.seqfolder.name)
 
-
-    def __del__(self):
-        self.cleanup()
+        #seqfolder = tempfile.mkdtemp(suffix='_timelapse')  # testing
+        #self.seqfolder_p = Path(seqfolder)
 
 
     def generate(self, video_file, file_list):