Try to fix: cannot schedule new futures after interpreter shutdown (#657

)

fixes #652 

https://docs.python.org/3.9/reference/datamodel.html#object.__del__
`It is not guaranteed that __del__() methods are called for objects that still exist when the interpreter exits.`

tensorboardX/record_writer.py

@@ -69,9 +69,11 @@ def __init__(self, path):
             raise ImportError("boto3 must be installed for S3 support.")
         self.path = path
         self.buffer = io.BytesIO()
+        self.closed = False
 
     def __del__(self):
-        self.close()
+        if not self.closed:
+            self.close()
 
     def bucket_and_path(self):
         path = self.path
@@ -94,6 +96,7 @@ def flush(self):
 
     def close(self):
         self.flush()
+        self.closed = True
 
 
 class S3RecordWriterFactory(object):

tests/test_embedding.py

@@ -86,3 +86,4 @@ def test_embedding_s3_mock(self):
                         label_img=all_images,
                         metadata_header=['digit', 'dataset'],
                         global_step=2)
+        w.close()