Skip to content

[BugFix] Fix tokenizer_client file name #7590

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
xiaoxiaohehe001 wants to merge 1 commit into
base: release/2.6
Choose a base branch
from
Open

[BugFix] Fix tokenizer_client file name #7590

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion fastdeploy/demo/tokenzier_client_demo.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@

import asyncio

from fastdeploy.input.tokenzier_client import (
from fastdeploy.input.tokenizer_client import (
Copy link

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

该 demo 文件名仍为 tokenzier_client_demo.py（拼写错误），但内部已改为导入 tokenizer_client。为避免后续用户/脚本按文件名搜索或引用时混淆，建议同时把 demo 文件名也更正为 tokenizer_client_demo.py（并相应更新任何引用/文档链接）。

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown

@PaddlePaddle-bot PaddlePaddle-bot Apr 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 建议 文件名本身仍含拼写错误

当前文件名为 tokenzier_client_demo.py（tokenzier 少了字母 i），本次 PR 仅修复了文件内的 import 路径，但未重命名文件本身。

建议同步将文件重命名为：

tokenizer_client_demo.py

AsyncTokenizerClient,
ImageDecodeRequest,
ImageEncodeRequest,
Expand Down
2 changes: 1 addition & 1 deletion fastdeploy/entrypoints/openai/response_processors.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
from typing import Any, Dict, List, Optional

from fastdeploy.entrypoints.openai.usage_calculator import count_tokens
from fastdeploy.input.tokenzier_client import AsyncTokenizerClient, ImageDecodeRequest
from fastdeploy.input.tokenizer_client import AsyncTokenizerClient, ImageDecodeRequest


class ChatResponseProcessor:
Expand Down
2 changes: 1 addition & 1 deletion fastdeploy/entrypoints/openai/v1/serving_chat.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@
OpenAiServingBase,
ServingResponseContext,
)
from fastdeploy.input.tokenzier_client import AsyncTokenizerClient, ImageDecodeRequest
from fastdeploy.input.tokenizer_client import AsyncTokenizerClient, ImageDecodeRequest
from fastdeploy.metrics.metrics import main_process_metrics
from fastdeploy.utils import api_server_logger
from fastdeploy.worker.output import LogprobsLists
Expand Down
0 fastdeploy/input/tokenzier_client.py → fastdeploy/input/tokenizer_client.py
View file
Open in desktop
File renamed without changes.
2 changes: 1 addition & 1 deletion tests/input/test_tokenizer_client.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
import pytest
import respx

from fastdeploy.input.tokenzier_client import (
from fastdeploy.input.tokenizer_client import (
Copy link

Copilot AI Apr 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR 描述里的 Modifications / Usage or Command / Accuracy Tests 等章节目前仍为空，同时 Checklist 也未勾选/说明。建议补充：具体改动点（例如重命名/新增 tokenizer_client.py 并更新引用）、如何验证（运行哪些单测/命令），以及如果不需要额外测试的原因，方便 reviewer/CI 追踪。

Copilot uses AI. Check for mistakes.
AsyncTokenizerClient,
ImageEncodeRequest,
VideoEncodeRequest,
Expand Down
Loading