Skip to content

Conversation

@ooooo-create
Copy link
Collaborator

No description provided.

@paddle-bot
Copy link

paddle-bot bot commented Oct 29, 2025

感谢你贡献飞桨文档,文档预览构建中,Docs-New 跑完后即可预览,预览链接:http://preview-pr-7591.paddle-docs-preview.paddlepaddle.org.cn/documentation/docs/zh/api/index_cn.html
预览工具的更多说明,请参考:飞桨文档预览工具

@SigureMo SigureMo changed the title [CI] explicit exit when pip install whl_url failed [CI] Explicit exit when pip install whl_url failed Nov 1, 2025
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

太有用了!喵的每次因为装 wheel 包出问题了他们都看不出来是啥问题,这些就应该 fast fail

@SigureMo
Copy link
Member

SigureMo commented Nov 1, 2025

这个影响的是 docs 这边的 CI 还是 Paddle 那边的 CI 还是都影响?

@ooooo-create
Copy link
Collaborator Author

这个影响的是 docs 这边的 CI 还是 Paddle 那边的 CI 还是都影响?

两边都会影响的,但是 paddle 那边我改成从本地 build 目录里安装了 PaddlePaddle/Paddle#76124 ,因为本地确实有,现在 paddle 那边应该不会触发这段逻辑啦


# Install logic:
# - If PADDLE_WHL is a .whl file URL: download and install locally
# - Otherwise: install directly via pip (supports package names and other URLs)
Copy link
Member

Choose a reason for hiding this comment

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

话说 pip 也支持 url 呀,为什么要分开处理呢?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

当时我看见这个耗时很离谱,就想先下载下来看看试试(感觉 wget 网络请求可能比 pip 处理的好

43f5f1b0dd74ceccc4bc1f7df3ffc553

Copy link
Member

Choose a reason for hiding this comment

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

嗯嗯可以

Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

LGTMeow 🐾

@SigureMo SigureMo merged commit 1b91071 into PaddlePaddle:develop Nov 1, 2025
4 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.

2 participants