Skip to content

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

Open

Description

@story645

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.

Activity

  1. anntzer commented on Nov 14, 2023

    @anntzer
    Contributor

    Why would we actually want to deprecate plt.xkcd?

  2. story645 commented on Nov 14, 2023

    @story645
    MemberAuthor

    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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions