Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

PoC: Document each rcParam individually #28930

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
Loading
from

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Oct 3, 2024

PR summary

Taking cues from the math symbol table, this adds a directive to generate a listing from the RcParams dictionary, with types taken from the validators (partially complete), though there could perhaps be other ways to implement it. Then the rc role is updated to point to these individual anchors.

We also have nowhere to put the documentation for each one yet, so the actual information is all just stubs. This would be the main obstacle to completing this change.

PR checklist

Taking cues from the math symbol table, this adds a directive to
generate a listing from the `RcParams` dictionary, with types taken from
the validators (partially complete), though there could perhaps be other
ways to implement it. Then the `rc` role is updated to point to these
individual anchors.

We also have nowhere to put the documentation for each one yet, so the
actual information is all just stubs.
@github-actions github-actions bot added topic: rcparams topic: sphinx extension Documentation: build building the docs Documentation: user guide files in galleries/users_explain or doc/users labels Oct 3, 2024
@timhoffm
Copy link
Member

timhoffm commented Oct 3, 2024

I believe we eventually should have a function-based/declarative approach to specifying rcParams, similar to argparse.

e.g.

config.add_param(""figure.dpi", type=float, default=100, help="figure dots per inch")

This would bind all relevant information together and make it easy to generate documentation. A minor obstacle would be to generate a reasonably formatted default matplotlibrc (but we could also leave the existing matplotlibrc for now and instead just write a function that checks that it's consistent with the config.

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

Successfully merging this pull request may close these issues.

2 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.