I do not understand how either the change to the context manager nor this change fixes it.

Before it was just setting the rcParams to the new xkcd values and not passing them to the context. I don't understand how it worked before ;-)

Ah, I understand. Before the clean up ofrc_context the previous state was cached on__init__, now it is stashed on__enter__.

This PR fixes the case of a context manager, but I suspect breaks if you just call

plt.xkcd()fig,ax=plt.subplots()...

Um yeah I guess so.

I find this a strange function in that it passes parameters to a style.

I’d actually propose an xkcd style and deprecating this function. If folks want to monkey with the randomness of the lines they can call the style and then do so directly with the rcparams.

I’m too dense to know how to make this work as both a context and directly changing the rcParams.

So you just need to add a call tocontext.__enter__() right after creating the context (in the old version of the code).
contextlib.ExitStack would make this easier to write :-)

Ok that’s easy.

xkcd style will not work because the rcparams machinery is messed up (or pick a less strong qualifier)#6157

frommatplotlibimportpatheffectscontext=rc_context()context.__enter__()

yields

Traceback (most recent call last):  File "testxkcd.py", line 3, in <module>    with plt.xkcd():  File "/Users/jklymak/anaconda3/envs/matplotlibdev/lib/python3.6/contextlib.py", line 83, in __enter__    raise RuntimeError("generator didn't yield") from NoneRuntimeError: generator didn't yield

Bah, we were doing something way too complex.

-    context = rc_context()-    try:-        rcParams['font.family'] = ['xkcd', 'Humor Sans', 'Comic Sans MS']-        rcParams['font.size'] = 14.0-        rcParams['path.sketch'] = (scale, length, randomness)-        rcParams['path.effects'] = [-            patheffects.withStroke(linewidth=4, foreground="w")]-        rcParams['axes.linewidth'] = 1.5-        rcParams['lines.linewidth'] = 2.0-        rcParams['figure.facecolor'] = 'white'-        rcParams['grid.linewidth'] = 0.0-        rcParams['axes.grid'] = False-        rcParams['axes.unicode_minus'] = False-        rcParams['axes.edgecolor'] = 'black'-        rcParams['xtick.major.size'] = 8-        rcParams['xtick.major.width'] = 3-        rcParams['ytick.major.size'] = 8-        rcParams['ytick.major.width'] = 3-    except:-        context.__exit__(*sys.exc_info())-        raise-    return context+    return rc_context({+        'font.family': ['xkcd', 'Humor Sans', 'Comic Sans MS'],+        'font.size': 14.0,+        'path.sketch': (scale, length, randomness),+        'path.effects': [patheffects.withStroke(linewidth=4, foreground="w")],+        'axes.linewidth': 1.5,+        'lines.linewidth': 2.0,+        'figure.facecolor': 'white',+        'grid.linewidth': 0.0,+        'axes.grid': False,+        'axes.unicode_minus': False,+        'axes.edgecolor': 'black',+        'xtick.major.size': 8,+        'xtick.major.width': 3,+        'ytick.major.size': 8,+        'ytick.major.width': 3,+    })

works (I tried it!).

I thought thats what I did ?

... sorry, not paying attention. back to the drawing board.

Yeah, that doesn't work for@tacaswell case...

Sorry to be dense about the context management. I can close this PR and let you work on it if easier.

FWIW,@tacaswell case isn't actually documented or tested anywhere! Its implied in the docstring.

this actually works in both cases

    from contextlib import ExitStack    stack = ExitStack    stack.enter_context(rc_context({        'font.family': ['xkcd', 'Humor Sans', 'Comic Sans MS'], ...    })    return stack

(except that it needs a backport of contextlib.ExitStack (or similar) on AncientPython, e.g.https://pypi.python.org/pypi/contextlib2/0.5.5).

Basically we want to enter the rc_context but returnanother context that does nothing on enter but pops the rc_context on exit. ExitStack exactly provides that functionality.

As pointed out by@tacaswell#8962 indeed broke backcompat for whoever was usingrc_context(...)without entering the context as a synonym forrc(...). I can't say I have much sympathy for this use case.

xkcd style will not work because the rcparams machinery is messed up (or pick a less strong qualifier)

Is this becausepath.effects wants a function, and there is no way to enter a function in a style sheet?

Is it possible to have a style-sheet parser fcn that reads a string frompath.effects and makes sure it is calling something frompatheffects before setting it?

OK, I suggest this change is consistent w/ the most documentation for now.

I suggest a way forward is to parse style sheets for strings that have patheffect calls, but to explicitly callpatheffect.XX ourselves to prevent security risks. There may be a fancier way to do that. But then folks could juststyles.use('xkcd') like any other style. See#6157

Closing in lieu of#9603

Thanks for your work on this@jklymak !

No problem -@anntzer solution is very cool!