🌐 AI搜索 & 代理 主页
Skip to content

DOC: Dynamically resize mathtext symbol tables (#26143) #30661

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.

Sign up for GitHub

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

Jump to bottom
Open
Ashish-coder-gif wants to merge 1 commit into
base: main
Choose a base branch
from
Open

DOC: Dynamically resize mathtext symbol tables (#26143) #30661

Ashish-coder-gif wants to merge 1 commit into from

Conversation

@Ashish-coder-gif
Copy link

@Ashish-coder-gif Ashish-coder-gif commented Oct 18, 2025

PR summary

PR checklist

PR Summary

This pull request updates doc/sphinxext/math_symbol_table.py to dynamically resize mathtext symbol tables during documentation builds.

Why this change is necessary

  • Improves maintainability of math symbol documentation generation.
  • Fixes minor formatting and whitespace issues.

What problem does it solve

  • Ensures consistent rendering of math symbols in the HTML docs.
  • Resolves warnings related to unused imports and blank lines.

Verification

  • Successfully built documentation locally using make -C doc html.
  • All pre-commit hooks passed after fixes.

This PR addresses issue #26143.

@github-actions github-actions bot added the Documentation: build building the docs label Oct 18, 2025
rcomer
rcomer requested changes Oct 18, 2025
View reviewed changes
Copy link
Member

@rcomer rcomer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the contribution @Ashish-coder-gif. Unfortunately the symbols tables are now not rendering at all. Do you know why?

Before this change:
https://matplotlib.org/devdocs/users/explain/text/mathtext.html#symbols

After this change:
https://output.circle-artifacts.com/output/job/cc17ec9e-e133-4ca9-9a4a-ead61dbccd8b/artifacts/0/doc/build/html/users/explain/text/mathtext.html#symbols

Note also, to solve the issue, the number of columns should change with window size. From a quick look at your code, I do not think you have implemented that.

@QuLogic QuLogic linked an issue Oct 23, 2025 that may be closed by this pull request
[Doc]: dynamically resize the mathtext symbol tables #26143
Open
@Ashish-coder-gif
Copy link
Author

Ashish-coder-gif commented Oct 26, 2025

kindly allow me to merge

@rcomer
Copy link
Member

rcomer commented Oct 26, 2025

Hi @Ashish-coder-gif, unfortunately the symbols table is still not rendering after your most recent commit. You can check this yourself by clicking “View the built docs” in the automated checks below, then navigate to User Guide -> Text -> Writing mathematical expressions, and scroll to the bottom of the page.

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

Reviewers

@rcomer rcomer rcomer requested changes

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

Documentation: build building the docs

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Doc]: dynamically resize the mathtext symbol tables

2 participants

@Ashish-coder-gif @rcomer