Closed stephentoub closed 4 months ago
This ports the tweaks in https://github.com/openai/tiktoken/pull/234. I noticed the differences as they also show up in the source for https://www.youtube.com/watch?v=zduSFxRajkE.
@tarekgh, if this conflicts with any of your changes, feel free to close this and I can re-make them after your changes land.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 68.83%. Comparing base (a139371) to head (849097e).
a139371
849097e
This ports the tweaks in https://github.com/openai/tiktoken/pull/234. I noticed the differences as they also show up in the source for https://www.youtube.com/watch?v=zduSFxRajkE.
@tarekgh, if this conflicts with any of your changes, feel free to close this and I can re-make them after your changes land.