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

Skip to content

[MNT]: Unify tex2uni #26029

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

Closed
oscargus opened this issue Jun 1, 2023 · 0 comments · Fixed by #26193
Closed

[MNT]: Unify tex2uni #26029

oscargus opened this issue Jun 1, 2023 · 0 comments · Fixed by #26193

Comments

@oscargus
Copy link
Member

oscargus commented Jun 1, 2023

Summary

tex2uni in _mathtext_data.py is now an unsorted dict with a mix of hex and decimal entries. This makes it quite hard to get an overview of.

Proposed fix

It would probably make sense to select one of the formats and sort it either on the number or the latex string.

An option would be to use the Unicode name which can be obtained as unicodedata.name(chr(8830)). In this case, sorting on the LaTeX-string may make the most sense. (One will have to modify the code a bit here as well as an integer is currently assumed.)

(In all circumstances, I'd suggest to write some code to do the conversion, not to do it manually, and also wait until #25933, #26010, and #26024 are merged.)

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

Successfully merging a pull request may close this issue.

2 participants