-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Sort tex2uni data in mathtext #26193
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
Conversation
Sure this can wait for merging. I did use a script, I will update the code with the comment. |
I think that this can be updated now. |
'triangle' : 9651, | ||
'iiiint' : 10764, | ||
'amalg' : 10815, | ||
'merge' : 10837, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is one fewer line in the new code, is something missing? (perhaps one of the things added by a recent PR?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well spotted! I think that there is a duplicate in the above list.
In current main len(tex2uni)
is 632 which is 1107-476+1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And it wasn't that hard to find...
'nsqsubseteq' : 8930,
'nsqsupseteq' : 8931,
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...but isn't it different: nsqSUBseteq
and nsqSUPseteq
(?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is the duplicate entry in main:
matplotlib/lib/matplotlib/_mathtext_data.py
Line 680 in 56732ff
'amalg' : 10815, |
matplotlib/lib/matplotlib/_mathtext_data.py
Line 1098 in 56732ff
'amalg' : 10815, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, you right. I actually read it a few times...
…193-on-v3.8.x Backport PR #26193 on branch v3.8.x (Sort tex2uni data in mathtext)
PR summary
Fixes #26029
Entries are converted to hex and the list is sorted on the LaTeX-strings
PR checklist