[XPU] Update xtdk version to 'latest' in download script#7632
Conversation
|
Thanks for your contribution! |
|
/skip-ci ci_iluvatar |
| version_xvllm="latest" | ||
| version_xtdk="3.6.2.1" | ||
| version_xtdk="latest" | ||
| fi |
There was a problem hiding this comment.
这里把 develop 分支的 version_xtdk 改成了 "latest"，会导致依赖不可复现（同一提交在不同时间拉到的 XTDK 可能不同），并且当前 CI（.github/workflows/_build_xpu.yml）会调用该脚本的 develop 模式，可能引入 CI 随时间波动/突然失败的风险。建议：默认仍然固定到一个已验证的版本号（或日期版本），并通过环境变量/额外参数显式允许使用 latest；或者至少在下载后打印/记录实际解析到的版本，便于排查问题。
PaddlePaddle-bot
left a comment
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-04-27 20:58:26
📋 Review 摘要
PR 概述：将 XPU 自定义算子依赖下载脚本中 xtdk 的非 stable 版本从固定版本号 3.6.2.1 改为 latest，与 xvllm 保持一致。
变更范围：custom_ops/xpu_ops/download_dependencies.sh
影响面 Tag：[XPU]
📝 PR 规范检查
PR 标题 Tag [XPU] 合规，但 Motivation 和 Modifications 两个核心段落均为空，未说明变更目的和具体改动。
标题建议（可直接复制）：
[XPU] Update xtdk version to 'latest' in download script
PR 描述建议（可直接复制，必须复刻 checklist §D2 模板的完整结构）：
## Motivation
将 XPU 自定义算子依赖下载脚本（download_dependencies.sh）中，非 stable 分支的 `xtdk` 版本从固定版本 `3.6.2.1` 更新为 `latest`，使其与同分支的 `xvllm` 版本策略保持一致，便于持续跟进最新的 xtdk 工具链。
## Modifications
- `custom_ops/xpu_ops/download_dependencies.sh`：将 `else` 分支中 `version_xtdk="3.6.2.1"` 改为 `version_xtdk="latest"`
## Usage or Command
N/A
## Accuracy Tests
N/A
## Checklist
- [x] Add at least a tag in the PR title.
- [ ] Format your code, run `pre-commit` before commit.
- [ ] Add unit tests. Please write the reason in this PR if no unit tests.
- [ ] Provide accuracy results.
- [ ] If the current PR is submitting to the `release` branch, make sure the PR has been submitted to the `develop` branch, then cherry-pick it to the `release` branch with the `[Cherry-Pick]` PR tag.问题
| 级别 | 文件 | 概述 |
|---|---|---|
| ❓ 疑问 | custom_ops/xpu_ops/download_dependencies.sh:19 |
使用 latest 可能导致构建不可重现 |
总体评价
改动简洁，逻辑一致。建议作者确认使用 latest 是否为预期策略，并补充 PR 描述说明变更动机。
| else | ||
| version_xvllm="latest" | ||
| version_xtdk="3.6.2.1" | ||
| version_xtdk="latest" |
There was a problem hiding this comment.
❓ 疑问 将 version_xtdk 设置为 latest 后，每次执行下载脚本可能获取不同版本的 xtdk，导致构建结果不可重现（non-reproducible build）。
若此为预期行为（如 CI 中始终跟进最新版本），建议在 PR 描述中说明；若需要可重现构建，建议在 stable 外还维护一个已知可用的固定版本号。
CI报告基于以下代码生成（30分钟更新一次）: 1 任务总览所有 required 任务已通过（2 通过 + 8 跳过/bypass），有 3 个可选任务失败（不阻塞合并），建议通过。
2 任务状态汇总2.1 Required任务 : 2/10 通过
2.2 可选任务 — 23/28 通过
3 失败详情（仅 required）无 required 失败任务。 |
|
/skip-ci ci_iluvatar |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #7632 +/- ##
=========================================
Coverage ? 0.00%
=========================================
Files ? 467
Lines ? 65031
Branches ? 0
=========================================
Hits ? 0
Misses ? 65031
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Motivation
Modifications
Usage or Command
Accuracy Tests
Checklist
[FDConfig],[APIServer],[Engine],[Scheduler],[PD Disaggregation],[Executor],[Graph Optimization],[Speculative Decoding],[RL],[Models],[Quantization],[Loader],[OP],[KVCache],[DataProcessor],[BugFix],[Docs],[CI],[Optimization],[Feature],[Benchmark],[Others],[XPU],[HPU],[GCU],[DCU],[Iluvatar],[Metax]]pre-commitbefore commit.releasebranch, make sure the PR has been submitted to thedevelopbranch, then cherry-pick it to thereleasebranch with the[Cherry-Pick]PR tag.