search

rasbt / LLMs-from-scratch

Implement a ChatGPT-like LLM in PyTorch from scratch, step by step
https://www.amazon.com/Build-Large-Language-Model-Scratch/dp/1633437167
Other
34.13k stars 4.18k forks source link

Fix incorrect argument name in LlamaTokenizer constructor #421

Closed rohanwinsor closed 1 month ago

rohanwinsor commented 1 month ago

This PR addresses an oversight in the LlamaTokenizer class by changing the constructor argument from filepath to tokenizer_file. This correction clarifies the purpose of the argument and enhances code readability.

review-notebook-app[bot] commented 1 month ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB