REFACT/BREAK(conf, plot): better name for contextman

Author: ankostis

docs/source/arch.rst

@@ -368,7 +368,7 @@ Architecture
         configuration` - this term implies also any :ref:`plot-customizations`
         done on the installed plotter (such as `plot styles`).
 
-        Installation happens by calling one of :func:`.installed_plotter` or
+        Installation happens by calling one of :func:`.installed_plotter_plugged()` or
         :func:`.set_installed_plotter` functions.
 
         The **default** *installed plotter* is the plotter instance that this project

graphtik/__init__.py

@@ -16,7 +16,7 @@
     abort_run,
     debug,
     evictions_skipped,
-    execution_pool,
+    execution_pool_plugged,
     get_execution_pool,
     is_abort,
     is_debug,
@@ -39,7 +39,7 @@
     tasks_marshalled,
 )
 
-## SEE ALSO: `.plot.installed_plotter()`, `.plot.set_installed_plotter()` &
+## SEE ALSO: `.plot.installed_plotter_plugged()`, `.plot.set_installed_plotter()` &
 #  `.plot.get_installed_plotter()` configs, not imported, unless plot is needed..
 
 from .modifiers import *  # noqa, on purpose to include any new modifiers

graphtik/config.py

@@ -99,7 +99,7 @@ def is_abort():
 
 
 @contextmanager
-def execution_pool(pool: "Optional[Pool]"):
+def execution_pool_plugged(pool: "Optional[Pool]"):
     """Like :func:`set_execution_pool()` as a context-manager to reset old value. """
     resetter = _execution_pool.set(pool)
     try:

graphtik/plot.py

@@ -1049,7 +1049,7 @@ def supported_plot_formats() -> List[str]:
 
 
 @contextmanager
-def installed_plotter(plotter: Plotter) -> None:
+def installed_plotter_plugged(plotter: Plotter) -> None:
     """Like :func:`set_installed_plotter()` as a context-manager to reset old value. """
     if not isinstance(plotter, Plotter):
         raise ValueError(f"Cannot install invalid plotter: {plotter}")

test/test_graphtik.py

@@ -25,7 +25,7 @@
     abort_run,
     compose,
     evictions_skipped,
-    execution_pool,
+    execution_pool_plugged,
     get_execution_pool,
     is_marshal_tasks,
     network,
@@ -85,7 +85,7 @@ def exemethod(request):
             else:
                 pool = mp_dummy.Pool(nsharks)
 
-            with execution_pool(pool), pool:
+            with execution_pool_plugged(pool), pool:
                 yield parallel
         else:
             yield parallel
@@ -1253,7 +1253,7 @@ def test_multithreading_plan_execution():
         )(partial(abspow, p=3)),
     )
 
-    with mp_dummy.Pool(int(2 * cpu_count())) as pool, execution_pool(pool):
+    with mp_dummy.Pool(int(2 * cpu_count())) as pool, execution_pool_plugged(pool):
         pool.map(
             # lambda i: graph.withset(name='graph').compute(
             lambda i: graph.compute(

test/test_plot.py

@@ -13,7 +13,12 @@
 from graphtik import base, compose, network, operation, plot
 from graphtik.modifiers import optional
 from graphtik.netop import NetworkOperation
-from graphtik.plot import Plotter, Style, get_installed_plotter, installed_plotter
+from graphtik.plot import (
+    Plotter,
+    Style,
+    get_installed_plotter,
+    installed_plotter_plugged,
+)
 
 
 @pytest.fixture
@@ -346,7 +351,7 @@ def test_plotter_customizations(pipeline, monkeypatch):
 
     ## New installed_plotter
     #
-    with installed_plotter(Plotter(style=Style(kw_legend={"URL": None}))):
+    with installed_plotter_plugged(Plotter(style=Style(kw_legend={"URL": None}))):
         dot = str(pipeline.plot())
         assert url not in dot
 
@@ -360,7 +365,7 @@ def test_plotter_customizations(pipeline, monkeypatch):
     assert url in dot
 
     url2 = "http://example.2.org"
-    with installed_plotter(Plotter(style=Style(kw_legend={"URL": url2}))):
+    with installed_plotter_plugged(Plotter(style=Style(kw_legend={"URL": url2}))):
         dot = str(pipeline.plot())
         assert url2 in dot
         assert url not in dot