Skip to content

fix(speculators): 移除 prepare-data 后对 token_freq.pt 的冗余校验 - #248

Open
Devin-DC-Huang wants to merge 1 commit into
Ascend:mainfrom
Devin-DC-Huang:fix/speculators-prepare-data-token-freq
Open

Devin-DC-Huang wants to merge 1 commit into
Ascend:mainfrom
Devin-DC-Huang:fix/speculators-prepare-data-token-freq

Conversation

@Devin-DC-Huang

Copy link
Copy Markdown
Contributor

上游 vllm-project/speculators v0.9.0 将 --skip-token-freq 默认改为 True,prepare-data 默认不再写 token_freq.pt;本 quick start 训练 不使用 reduced vocabulary,token_freq.pt 仅作「数据已落盘」校验。
改为只校验数据集行数(load_from_disk),删掉对 token_freq.pt 的
torch.load 校验。

上游 vllm-project/speculators v0.9.0 将 --skip-token-freq 默认改为
True,prepare-data 默认不再写 token_freq.pt;本 quick start 训练
不使用 reduced vocabulary,token_freq.pt 仅作「数据已落盘」校验。
改为只校验数据集行数(load_from_disk),删掉对 token_freq.pt 的
torch.load 校验。
@github-actions

Copy link
Copy Markdown
Contributor

📖 文档预览:https://ascend.github.io/docs/pr-preview/pr-248/sources/speculators/

也可打开整站首页。

构建自 62f00864911b36b573b6e3afd9f939c396ac51f4。
每次推送后自动更新,PR 关闭时自动清理。

github-actions Bot pushed a commit that referenced this pull request Oct 20, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant