Skip to content

[XPU] [CI] Change Paddle Version to Nightly#5346

Merged
plusNew001 merged 13 commits intoPaddlePaddle:developfrom
plusNew001:xpu-ci-update
Dec 5, 2025
Merged

[XPU] [CI] Change Paddle Version to Nightly#5346
plusNew001 merged 13 commits intoPaddlePaddle:developfrom
plusNew001:xpu-ci-update

Conversation

@plusNew001
Copy link
Collaborator

Motivation

💡 If this PR is a Cherry Pick, the PR title needs to follow the format by adding the [Cherry-Pick] label at the very beginning and appending the original PR ID at the end. For example, [Cherry-Pick][CI] Add check trigger and logic(#5191)

💡 如若此PR是Cherry Pick,PR标题需遵循格式,在最开始加上[Cherry-Pick]标签,以及最后面加上原PR ID,例如[Cherry-Pick][CI] Add check trigger and logic(#5191)

Modifications

Usage or Command

Accuracy Tests

Checklist

  • Add at least a tag in the PR title.
    • Tag list: [[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]]
    • You can add new tags based on the PR content, but the semantics must be clear.
  • 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.

Copilot AI review requested due to automatic review settings December 3, 2025 03:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR changes the PaddlePaddle-XPU installation method in the pytest CI script from a specific locked nightly build (dated 20251123) to using the latest available nightly version via the official package repository.

Key Changes:

  • Modified installation command to use --pre paddlepaddle-xpu from the nightly repository instead of a direct wheel URL
  • Aligns with the documented installation pattern shown in the project's documentation

# 安装PaddlePaddle
echo "安装PaddlePaddle..."
python -m pip install https://paddle-whl.bj.bcebos.com/nightly/xpu-p800/paddlepaddle-xpu/paddlepaddle_xpu-3.3.0.dev20251123-cp310-cp310-linux_x86_64.whl
python -m pip install --pre paddlepaddle-xpu -i https://www.paddlepaddle.org.cn/packages/nightly/xpu-p800/
Copy link

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

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

根据自定义审查指南,PR描述信息不完整。描述中缺少关键信息:

  1. 动机(Motivation)部分为空: 需要说明为什么要将此CI脚本从固定版本的paddlepaddle-xpu改为使用最新的nightly版本。
  2. 修改内容(Modifications)部分为空: 需要详细说明具体做了哪些修改。
  3. 缺少与scripts/run_ci_xpu.sh的对比说明: 在scripts/run_ci_xpu.sh文件的第49行有注释"由于ep并行报错暂时锁死paddle版本",该脚本仍然使用固定版本。需要解释为什么run_xpu_ci_pytest.sh可以使用最新nightly版本,而run_ci_xpu.sh不能。

建议补充完整的PR描述,说明:

  • 为什么需要这个修改
  • 这个修改解决了什么问题
  • 为什么pytest CI可以使用最新nightly版本而不会遇到EP并行报错的问题

Copilot generated this review using guidance from repository custom instructions.
Comment on lines +77 to 78
python -m pip install --pre paddlepaddle-xpu -i https://www.paddlepaddle.org.cn/packages/nightly/xpu-p800/

Copy link

Copilot AI Dec 3, 2025

Choose a reason for hiding this comment

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

这个变更从固定版本的paddlepaddle-xpu切换到最新的nightly版本,可能会引入CI稳定性问题。建议:

  1. 添加注释说明: 在这行代码上方添加注释,解释为什么这个CI脚本使用nightly版本是安全的(与scripts/run_ci_xpu.sh的区别)。
  2. 考虑版本验证: 在安装后添加版本验证步骤,记录安装的实际版本号,便于在CI失败时快速定位是否与版本变化有关。

例如:

# 安装PaddlePaddle
echo "安装PaddlePaddle..."
# pytest CI不涉及EP并行,可以使用最新nightly版本
python -m pip install --pre paddlepaddle-xpu -i https://www.paddlepaddle.org.cn/packages/nightly/xpu-p800/
# 记录实际安装的版本
python -c "import paddle; print(f'PaddlePaddle version: {paddle.__version__}')"
Suggested change
python -m pip install --pre paddlepaddle-xpu -i https://www.paddlepaddle.org.cn/packages/nightly/xpu-p800/
# pytest CI不涉及EP并行,此处可安全使用最新nightly版本,与scripts/run_ci_xpu.sh不同
python -m pip install --pre paddlepaddle-xpu -i https://www.paddlepaddle.org.cn/packages/nightly/xpu-p800/
# 记录实际安装的PaddlePaddle版本,便于CI问题定位
python -c "import paddle; print(f'PaddlePaddle version: {paddle.__version__}')"

Copilot uses AI. Check for mistakes.
@codecov-commenter
Copy link

codecov-commenter commented Dec 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (develop@7b0b6e4). Learn more about missing BASE report.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #5346   +/-   ##
==========================================
  Coverage           ?   59.25%           
==========================================
  Files              ?      325           
  Lines              ?    40220           
  Branches           ?     6089           
==========================================
  Hits               ?    23833           
  Misses             ?    14553           
  Partials           ?     1834           
Flag Coverage Δ
GPU 59.25% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@paddle-bot
Copy link

paddle-bot bot commented Dec 3, 2025

Thanks for your contribution!

@plusNew001 plusNew001 merged commit b5a7abe into PaddlePaddle:develop Dec 5, 2025
14 of 17 checks passed
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.

3 participants