Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[ENH]: add path.sketch.{scale, length, randomness, seed} rcParams #27318

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
story645 opened this issue Nov 13, 2023 · 2 comments
Open

[ENH]: add path.sketch.{scale, length, randomness, seed} rcParams #27318

story645 opened this issue Nov 13, 2023 · 2 comments

Comments

@story645
Copy link
Member

Problem

Looking at #25796 and #26854, it might be helpful to be able to independently set the different parts of sketch and not just as a tuple. This is motivated by wanting to make plt.xkcd discouraged and wanting the cleanest alternative for passing in the sketch params.

Proposed solution

#25796 moves in this direction by allowing both

mpl.rcParams['path.sketch'] = (scale, length, randomness, seed)
mpl.rcParams[`path.sketch_seed'] = seed

and if we're OK w/ the other parameters getting their own rcParams then they can have a unified naming scheme. Alternatively, we can probably do the parsing so None = default but I don't like having to remember which position is which param.

@anntzer
Copy link
Contributor

anntzer commented Nov 14, 2023

Why would we actually want to deprecate plt.xkcd?

@story645
Copy link
Member Author

Why would we actually want to deprecate plt.xkcd

Consensus on the call is we won't, I think we should b/c I think all the styling functionality should just live together & it's odd to have this one special cased styling function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants