Skip to content

Commit

Permalink
docs: fix typo in convert.py (#17)
Browse files Browse the repository at this point in the history
signficant -> significant
  • Loading branch information
eltociear authored Aug 21, 2023
1 parent a227226 commit cb01ade
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion models/convert.py
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ def bytes_to_unicode():
The reversible bpe codes work on unicode strings.
This means you need a large # of unicode characters in your vocab if you want to avoid UNKs.
When you're at something like a 10B token dataset you end up needing around 5K for decent coverage.
This is a signficant percentage of your normal, say, 32K bpe vocab.
This is a significant percentage of your normal, say, 32K bpe vocab.
To avoid that, we want lookup tables between utf-8 bytes and unicode strings.
And avoids mapping to whitespace/control characters the bpe code barfs on.
"""
Expand Down

0 comments on commit cb01ade

Please sign in to comment.