Skip to content

Commit

Permalink
Fix: unnecessary warning issue in HuggingFace LLM when tokenizer is p…
Browse files Browse the repository at this point in the history
…rovided as argument(run-llama#16035) (run-llama#16037)
  • Loading branch information
CuriousMonkey7 authored Sep 16, 2024
1 parent 6816ad9 commit f037de8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -259,9 +259,9 @@ def __init__(
tokenizer_name, **tokenizer_kwargs
)

if tokenizer.name_or_path != model_name:
if tokenizer.name_or_path != model.name_or_path:
logger.warning(
f"The model `{model_name}` and tokenizer `{tokenizer.name_or_path}` "
f"The model `{model.name_or_path}` and tokenizer `{tokenizer.name_or_path}` "
f"are different, please ensure that they are compatible."
)

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ exclude = ["**/BUILD"]
license = "MIT"
name = "llama-index-llms-huggingface"
readme = "README.md"
version = "0.3.3"
version = "0.3.4"

[tool.poetry.dependencies]
python = ">=3.8.1,<4.0"
Expand Down

0 comments on commit f037de8

Please sign in to comment.