NotificationsYou must be signed in to change notification settings
Fork7.9k
Star21.4k

Correct Unicode for [lg]napprox#26010

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Merged

ksunden merged 1 commit intomatplotlib:mainfromoscargus:napprox

Jun 14, 2023

Merged

Correct Unicode for [lg]napprox#26010

ksunden merged 1 commit intomatplotlib:mainfromoscargus:napprox

Jun 14, 2023

Conversation

Copy link

Member

oscargus commentedMay 30, 2023

PR summary

While checking the output of#25933 I realized that these two used the wrong code point (prec/succnapprox). (Visible in the current documentation as well.)

See the resulting docs, but to confirm that these are the correct ones:https://www.compart.com/en/unicode/U+2A89 https://www.compart.com/en/unicode/U+2A8A

PR checklist

"closes #0000" is in the body of the PR description tolink the related issue
new and changed code istested
Plotting related features are demonstrated in anexample
New Features andAPI Changes are noted with adirective and release note
Documentation complies withgeneral anddocstring guidelines

oscargus added the topic: text/mathtext label

May 30, 2023

timhoffm approved these changes

May 30, 2023

View reviewed changes

Copy link

Member

timhoffm left a comment

Choose a reason for hiding this comment

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

Do we want to add a release not to make people aware that this was incorrect and is now fixed, or is it just a fix of incorrect behavior that does not need mention.