Fix #10842 - try catch around plugin.setup and make sure to release t…

…he Python GIL
NREL · Dec 3, 2024 · 552f5dd · 552f5dd
1 parent 0c73d29
commit 552f5dd
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/EnergyPlus/PluginManager.cc b/src/EnergyPlus/PluginManager.cc
@@ -615,7 +615,12 @@ PluginManager::PluginManager(EnergyPlusData &state) : eplusRunningViaPythonAPI(s
 
     // IMPORTANT - CALL setup() HERE ONCE ALL INSTANCES ARE CONSTRUCTED TO AVOID DESTRUCTOR/MEMORY ISSUES DURING VECTOR RESIZING
     for (auto &plugin : state.dataPluginManager->plugins) {
-        plugin.setup(state);
+        try {
+            plugin.setup(state);
+        } catch (const FatalError &e) {
+            PyGILState_Release(gil);
+            throw e;
+        }
     }
 
     std::string const sGlobals = "PythonPlugin:Variables";