[API Compatibility] Add paddle.nn.init.sparse_()#79310
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #79310 +/- ##
===========================================
Coverage ? 100.00%
===========================================
Files ? 1
Lines ? 50
Branches ? 0
===========================================
Hits ? 50
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
| return init(tensor) | ||
|
|
||
|
|
||
| def sparse_( |
| ) | ||
| out_var._share_underline_tensor_to(var) | ||
| return None | ||
| return out_var |
There was a problem hiding this comment.
这个应该所有的 paddle.nn.init.* 都会有这个问题,其他同类API,如果也有这个问题就一起改了
There was a problem hiding this comment.
修改返回值的方式是否有效,似乎也不确定,我看XPU的dygraph测试一直没过
CI报告基于以下代码生成(30分钟更新一次): 1 Required任务 : 41/48 通过
2 失败详情🔴 Windows-OPENBLAS / Build and test — 不稳定问题(置信度: 高)错误类型: 不稳定问题 | 置信度: 高
关键日志:
修复建议:
关联变更: 未发现与本 PR 修改文件相关。 🔴 Model-Benchmark / Benchmark test — 环境问题(置信度: 高)错误类型: 环境问题 | 置信度: 高
关键日志:
修复建议:
关联变更: 未发现与本 PR 修改文件相关。 🔴 Slice / Slice test — 不稳定问题(置信度: 中)错误类型: 不稳定问题 | 置信度: 中
关键日志:
修复建议:
关联变更: 未发现与本 PR 修改文件相关。 |
|
XPU测试没过,似乎是dygraph设置元素为0的时候没有成功 不过这个我手头条件难以复现 |
|
/re-run all-failed |
| """ | ||
| init = Normal(mean=mean, std=std) | ||
|
|
||
| if in_dygraph_mode(): |
There was a problem hiding this comment.
其他的init API都处理下返回值,这个文件是后面新增的,一般无后向兼容问题
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-06-18 20:16:44
📋 Review 摘要
PR 概述:新增 paddle.nn.init.sparse_(),并让多组 paddle.nn.init.*_ 函数在动态图下返回输入 Tensor。
变更范围:python/paddle/nn/init.py、test/legacy_test/test_nn_init_function.py
影响面 Tag:[User Experience]
问题
未发现新的非历史行间问题。历史未解决项的当前状态见下方;PR 规范问题在下面章节报,不在这里重复。
历史 Findings 修复情况
| Finding | 问题 | 状态 |
|---|---|---|
| F1 | static graph 下 normal_() 返回 op 后继续当 Tensor 使用 |
|
| F2 | 新公开 API 文档/示例不完整 | 🔄 部分修复 |
| F3 | sparsity 缺少 [0, 1] 范围校验 |
|
| F4 | static graph 下切片赋值未按 paddle.static.setitem 重新绑定 |
📝 PR 规范检查
标题 Tag API Compatibility 不在 Paddle 模板枚举中;描述模板结构完整,精度变化字段已填写 否。
标题建议(可直接复制):
[User Experience] Add paddle.nn.init.sparse_()
总体评价
本轮按 Paddle checklist A3/B3 复核了公开 Python API、动态图/静态图行为、测试和文档入口。新增的非历史问题未发现,但历史的 static graph 语义和参数校验问题仍需修复后再合入。
|
/re-run all-failed |
1 similar comment
|
/re-run all-failed |
PR Category
User Experience
PR Types
Improvements
Description
paddle.nn.init.sparse_()paddle.nn.init.normal_()to return tensor in dygraph mode是否引起精度变化
否