Skip to content

upload cuda132 build whl#4754

Merged
zjjlivein merged 1 commit into
PaddlePaddle:developfrom
zjjlivein:add_cuda13_build_whl
Jul 8, 2026
Merged

upload cuda132 build whl#4754
zjjlivein merged 1 commit into
PaddlePaddle:developfrom
zjjlivein:add_cuda13_build_whl

Conversation

@zjjlivein

Copy link
Copy Markdown
Collaborator

Before submitting

  • Lint code. If there are lint issues, please format the code first.
# Install and register `pre-commit` in the project folder
pip install pre-commit && pre-commit install

# Process previous code files separately
pre-commit run --file XXXX.py
  • Add test cases into tests folder. If there are codecov issues, please add tests cases first.

PR types

PR changes

Description

@zjjlivein zjjlivein merged commit 88a1e6c into PaddlePaddle:develop Jul 8, 2026
6 of 7 checks passed

@risemeup1111 risemeup1111 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

已完成首轮检查,发现一个需要修复的发布路径问题,详情见行级评论。当前 CUDA 13.2 wheel 会上传到与现有 cu132 安装索引不一致的目录,建议修正后再视为发布链路可用。

P3 优先级:P3

非行级:PR 描述仍保留模板内容,缺少本次变更的目的、影响范围和验证方式。建议补充类似 “将 PaddleFormers nightly wheel 增加上传到 CUDA 13.2/cu132 包索引;验证方式:确认 build whl workflow 上传路径与 cu132 安装源一致” 的说明,方便后续追溯。

处理要求:请针对该评论进行回复(同意并已修改请回复 Done,不同意请说明理由)。

Powered by Nyanpasu with gpt-5.5 xhigh, please check the suggestions carefully.

continue
fi
for cuda in cu126 cu129 cu130; do
for cuda in cu126 cu129 cu130 cuda132; do

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 优先级:P1

问题: 这里新增的是 CUDA 13.2 的 wheel 发布目标,但仓库里的 CUDA 13.2 安装和回归入口使用的都是 cu132 目录,例如 scripts/regression/install_requirements.shCE_Develop_cu132_py312 / CE_Release_cu132_py312 都从 https://www.paddlepaddle.org.cn/packages/nightly/cu132/ 拉包。当前写成 cuda132 会把产物上传到 paddle-whl/nightly/cuda132/paddleformers/,下游按 cu132 索引安装时拿不到本次构建的 wheel。

影响: CUDA 13.2 nightly wheel 发布链路对用户和 CE 回归不可用。

期望修改: 将上传目标改成与现有包索引一致的 cu132

Suggested change
for cuda in cu126 cu129 cu130 cuda132; do
for cuda in cu126 cu129 cu130 cu132; do

处理要求:请针对该评论修复并提交新的 commit。

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.

2 participants