search

openai / tiktoken

tiktoken is a fast BPE tokeniser for use with OpenAI's models.
MIT License
11.8k stars 803 forks source link

Remove unused code #118

Closed laran closed 1 year ago

laran commented 1 year ago

There was no case when cache_dir would == "".

hauntsaninja commented 1 year ago

Thanks for the PR! This is currently used:

λ XYZ= python -c 'import os; print(repr(os.environ["XYZ"]))'
''
laran commented 1 year ago

Seems like if the environment variable is set to an empty string, it could be empty.