-
Notifications
You must be signed in to change notification settings - Fork 660
【Hackathon 9th No.109】[CppExtension] [XPU] Support build Custom OP in setuptools 80+ -part #5106
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Thanks for your contribution! |
好像失败了?
PaddlePaddle/Paddle#76008 (comment) 提到的 gpu 算子找不到是完全找不到嘛?我之前测的 case 是能跑起来的来着 |
我在 xpu 的 build.sh 打日志: 结果,连 tmp 目录都木有 所以日志里面提示找不到安装的内容也就很合理了 ~ 只不过,那么之前是怎么安装的?
build 的日志: 说明,算子正确的拷贝到 gpu 目录中了 ~ 并且,我在本地模拟测试了 从日志和本地测试来看,不应该出现算子找不到的问题 ~ 而那个 debug 的 pr 中的日志提示错误: 难道是这个算子有问题? 综上,我这里是搞不清楚为啥 debug 的那个 ci 会出问题了 ~ 难道是 ci 的设置的问题? 关联 #4998 |
|
#4998 我试了一下不 patch cpp_extension ,结果 CI_XPU 同样不行 ~
|
SigureMo
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我看 #4998 已经过了,gpu 那边还有问题嘛?没有问题的话 Paddle 那边也可以推进合入了~
要不就先合了吧 ~ 从日志上看应该没问题,出问题再定位吧,现在瞎猜也是浪费时间 ~ |
ok,我推进下 |
Motivation
关联 #4977
框架兼容 setuptools80+ 之后,cpp_extension 的打包方式发生了改变,因此需要修改 FD 中 copy_ops 的逻辑。
本 PR 修改 XPU 的 build 部分,以补充 #4977 修改。
Modifications
参考 #4977
Usage or Command
参考 #4977
Accuracy Tests
参考 #4977
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.@SigureMo