Warn when trying to start a GUI event loop out of the main thread.

anntzer · anntzer · commit b910442caed4 · 2019-10-24T22:29:52.000+02:00
diff --git a/lib/matplotlib/pyplot.py b/lib/matplotlib/pyplot.py
@@ -26,6 +26,10 @@
 import re
 import sys
 import time
+try:
+    import threading
+except ImportError:
+    import dummy_threading as threading
 
 from cycler import cycler
 import matplotlib
@@ -177,6 +181,12 @@ def findobj(o=None, match=None, include_self=True):
     return o.findobj(match, include_self=include_self)
 
 
+def _get_required_interactive_framework():
+    global _backend_mod
+    return getattr(
+        _backend_mod.FigureCanvas, "required_interactive_framework", None)
+
+
 def switch_backend(newbackend):
     """
     Close all open figures and set the Matplotlib backend.
@@ -224,8 +234,7 @@ class _backend_mod(matplotlib.backend_bases._Backend):
     _log.debug("Loaded backend %s version %s.",
                newbackend, _backend_mod.backend_version)
 
-    required_framework = getattr(
-        _backend_mod.FigureCanvas, "required_interactive_framework", None)
+    required_framework = _get_required_interactive_framework()
     if required_framework is not None:
         current_framework = cbook._get_running_interactive_framework()
         if (current_framework and required_framework
@@ -248,6 +257,9 @@ class _backend_mod(matplotlib.backend_bases._Backend):
 
 def new_figure_manager(*args, **kwargs):
     """Create a new figure manager instance."""
+    if (_get_required_interactive_framework()
+            and threading.current_thread() is not threading.main_thread()):
+        cbook._warn_external("Don't do this!")
     global _backend_mod
     return _backend_mod.new_figure_manager(*args, **kwargs)
 
@@ -276,6 +288,9 @@ def show(*args, **kwargs):
         This is experimental, and may be set to ``True`` or ``False`` to
         override the blocking behavior described above.
     """
+    if (_get_required_interactive_framework()
+            and threading.current_thread() is not threading.main_thread()):
+        cbook._warn_external("Don't do this!")
     global _backend_mod
     return _backend_mod.show(*args, **kwargs)
 
