-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
昇腾和寒武纪相关代码退场 Tracking Issue #52319
Comments
1-3. 清理`WITH_MLU`相关代码,删除 `paddle/fluid/platform/device/mlu`目录,删除 `paddle/fluid/operators/*_mlu.*` 文件。具体需清理的文件如下所示:
4. 清理 `is_compiled_with_mlu` 相关代码,具体如下:
5-7. 清理`WITH_ASCEND_CL PADDLE_WITH_ASCEND_CL WITH_ASCEND_CXX11 WITH_ASCEND_INT64`相关代码,删除 `paddle/fluid/operators/*_npu.*`,删除 `/paddle/fluid/platform/device/npu`目录。
8. 清理 `is_compiled_with_npu` 相关代码:
9. 清理 `PADDLE_WITH_ASCEND` 相关代码,具体如下:
10. 清理 `WITH_ASCEND` 相关代码,具体如下:
11. 清理 paddle_build.sh 脚本中所有的 WITH_MLU WITH_ASCEND* 关键字
|
遗留问题代码冲突
修改造成功能缺失
暂时不能完全去掉 npu 相关代码
|
@KimBioInfoStudio 确实缺少了 |
认领 2 |
@GreatV 同理, 还有 hccl |
@KimBioInfoStudio hccl这些应该会在清理 |
第一项任务中,这四个文件是不是也要删除 |
应该是的 |
是的,现在 mlu 走的是 decvice / custom device, 所以 mlu 整个 dir 都可以删除 |
应该不行,有些代码是在 Python 里面,WITH_ASCEND_CL 大部分都是在 cmake 或者 cc 里面 |
@ronny1996 @qili93 seems we should remove cnpai for mlu profiler, add redesign a new api for all custom_device cc @luotao1 |
@luotao1 @Liyulingyue in latest develop tools/infer_prune_patches/op_registry.h.patch:97: REGISTER_OP_KERNEL(op_type, MLU, ::paddle::platform::MLUPlace, __VA_ARGS__)
tools/infer_prune_patches/op_registry.h.patch:131: #define REGISTER_OP_MLU_KERNEL_FUNCTOR(op_type, ...) \
tools/infer_prune_patches/op_registry.h.patch:133: op_type, MLU, ::paddle::platform::MLUPlace, DEFAULT_TYPE, \ |
@KimBioInfoStudio @Liyulingyue 这个patch先不用改,是推理侧为了裁剪库体积写的脚本,不是简单删除即可。 |
Okay, 这个应该是需要重新取diff了,之前一致有个疑惑,删除custom device 代码会导致infer lib size 增加,现在大致猜测是因为 diff apply 不上去导致的 |
https://github.com/PaddlePaddle/Paddle/search?q=npu 还剩下最后一项【清理 NPU 相关代码】,还有不少npu代码。 |
@KimBioInfoStudio @Liyulingyue 整个 |
|
感谢 @KimBioInfoStudio 统计
ascend 关键字已检查完毕,剩余的都不需要删除
|
昇腾和寒武纪相关代码退场已全部完成,感谢参与的小伙伴们!
欢迎继续参与快乐开源的其他任务! |
一、 项目说明
用于跟踪 昇腾和寒武纪相关代码退场 并记录该计划下各个子任务的计划和进度。
二、 主要工作 (18/18)
paddle/fluid/platform/device/mlu
目录✅(2023/3/31)paddle/fluid/operators/*_mlu.*
文件✅(2023/4/3)WITH_MLU
相关代码✅(2023/4/13)is_compiled_with_mlu
相关代码✅(2023/4/3)paddle/fluid/operators/*_npu.*
、清理WITH_ASCEND_INT64
相关代码✅(2023/4/11)/paddle/fluid/platform/device/npu
目录✅(2023/4/1)WITH_ASCEND_CL PADDLE_WITH_ASCEND_CL WITH_ASCEND_CXX11
相关代码✅(2023/4/13)is_compiled_with_npu
相关代码✅(2023/4/6)PADDLE_WITH_ASCEND
相关代码✅(2023/4/13)WITH_ASCEND
相关代码✅(2023/4/10)paddle_build.sh
脚本中所有的WITH_MLU
WITH_ASCEND*
关键字✅(2023/3/31)CNCL
相关代码✅(2023/4/17)HCCL
相关代码✅(2023/4/20)is_mlu_place
is_npu_place
is_npu_pinned_place
SupportNPU
SupportMLU
相关代码✅(2023/4/28)需要清理的具体文件可参考此链接。
The text was updated successfully, but these errors were encountered: