Improvements per PR feedback

nv-morpheus · Sep 5, 2023 · 7b39722 · 7b39722
1 parent fbab666
commit 7b39722
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/python/mrc/tests/utils.cpp b/python/mrc/tests/utils.cpp
@@ -24,7 +24,6 @@
 #include <pybind11/gil.h>  // for gil_scoped_acquire
 #include <pybind11/pybind11.h>
 
-#include <array>  // std::array needed for py::print
 #include <sstream>
 #include <stdexcept>
 
@@ -37,10 +36,8 @@ struct RequireGilInDestructor
 {
     ~RequireGilInDestructor()
     {
-        // Grab the GIL to print
+        // Grab the GIL
         py::gil_scoped_acquire gil;
-
-        py::print("RequireGilInDestructor::~RequireGilInDestructor() - calling destructor");
     }
 };
 

diff --git a/python/tests/test_gil_tls.py b/python/tests/test_gil_tls.py
@@ -21,15 +21,13 @@
 TLS = threading.local()
 
 
-def test_gc_called_in_thread_finalizer():
+def test_gil_thread_local_storage():
     """
     Test to reproduce issue #362
     No asserts needed if it doesn't segfault, then we're good
     """
-    mrc.logging.log("Building pipeline")
 
     def source_gen():
-        mrc.logging.log("source_gen")
         x = RequireGilInDestructor()
         TLS.x = x
         yield x