[HPU][CI]Hpu ci update #5116
[HPU][CI]Hpu ci update #5116
Conversation
Set pip global index URL to Tsinghua mirror and uninstall PaddleCustomDevice and fastdeploy.
|
Thanks for your contribution! |
There was a problem hiding this comment.
Pull Request Overview
This PR reorganizes the pip configuration setup in the HPU CI script by moving the pip index URL configuration to execute before installing requirements, rather than after uninstalling packages. This ensures the correct PyPI mirror is used for all subsequent pip installations.
Key Changes
- Moved pip index-url configuration to line 18, before
requirements.txtinstallation - Removed duplicate pip index-url configuration from line 26 (previously after uninstall commands)
| lsof -t -i :$FD_API_PORT | xargs kill -9 || true | ||
|
|
||
| echo "pip requirements" | ||
| python -m pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple |
There was a problem hiding this comment.
The PR title "[HPU][CI]Hpu ci update" lacks a clear description of what is being updated. According to the project guidelines, the title should follow the format [CLASS]Title with a descriptive summary. Consider changing it to something like "[HPU][CI]Move pip mirror configuration before requirements installation" to better describe the specific change being made.
| lsof -t -i :$FD_API_PORT | xargs kill -9 || true | ||
|
|
||
| echo "pip requirements" | ||
| python -m pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple |
There was a problem hiding this comment.
The PR description is incomplete. According to the project guidelines, the description should at minimum explain why these modifications are being made and what problem is being solved. Please add details about why moving the pip configuration earlier in the script is necessary (e.g., to ensure requirements.txt dependencies are downloaded from the Tsinghua mirror for better performance/reliability in the CI environment).
1 participant
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.