Skip to content

correct roman numeral VII description#15523

Merged
ethanfurman merged 1 commit into
masterfrom
ethanfurman-unicode-howto
Aug 26, 2019
Merged

correct roman numeral VII description#15523
ethanfurman merged 1 commit into
masterfrom
ethanfurman-unicode-howto

Conversation

@ethanfurman

Copy link
Copy Markdown
Member

No description provided.

@gnprice

gnprice commented Aug 27, 2019

Copy link
Copy Markdown
Contributor

Hmm, is this right? I think U+2167 really is ROMAN NUMERAL EIGHT.

Here's a table:

>>> import unicodedata
>>> for i in range(0x2160, 0x2170): print("U+{:X} {}  {}".format(i, chr(i), unicodedata.name(chr(i))))
... 
U+2160 Ⅰ  ROMAN NUMERAL ONE
U+2161 Ⅱ  ROMAN NUMERAL TWO
U+2162 Ⅲ  ROMAN NUMERAL THREE
U+2163 Ⅳ  ROMAN NUMERAL FOUR
U+2164 Ⅴ  ROMAN NUMERAL FIVE
U+2165 Ⅵ  ROMAN NUMERAL SIX
U+2166 Ⅶ  ROMAN NUMERAL SEVEN
U+2167 Ⅷ  ROMAN NUMERAL EIGHT
U+2168 Ⅸ  ROMAN NUMERAL NINE
U+2169 Ⅹ  ROMAN NUMERAL TEN
U+216A Ⅺ  ROMAN NUMERAL ELEVEN
U+216B Ⅻ  ROMAN NUMERAL TWELVE
U+216C Ⅼ  ROMAN NUMERAL FIFTY
U+216D Ⅽ  ROMAN NUMERAL ONE HUNDRED
U+216E Ⅾ  ROMAN NUMERAL FIVE HUNDRED
U+216F Ⅿ  ROMAN NUMERAL ONE THOUSAND

... I think I see the issue. We had the right name, but the wrong real live character 🙂 The character in the file is Ⅶ... aka U+2166 ROMAN NUMERAL SEVEN.

I'll send a quick PR to fix it.

@gnprice

gnprice commented Aug 27, 2019

Copy link
Copy Markdown
Contributor

Sent as #15541 .

lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
websurfer5 pushed a commit to websurfer5/cpython that referenced this pull request Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants