已合并
add test for nn.Module.npu() #35163
zf_zhang创建于 5月9日
add test for nn.Module.npu() #35163
已合并
Pull Request已成功合入, 合并人@ascend-robot
(感谢 zf_zhang 的贡献)ascend-robot
5月9日 评论:
5月9日 评论:
ascend-robot
5月9日 评论:
5月9日 评论:
Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| test | ✅ 李伟, sunyu-xuan (2/2) | ✅ 李伟 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
zf_zhang, thanks for your pull request. All authors of the commits have signed the CLA. 👍


5月9日 添加了label:ascend-cla/yes
zf_zhang
5月9日 评论:
5月9日 评论:
compile


此处折叠了140条消息 查看更多
5月12日 修改了pull request 的描述
5月12日 添加了label:approvedlgtm
ascend-robot
5月12日 评论:
5月12日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: 李伟.
Reviewers who wrote a comment of /lgtm are: 李伟, sunyu-xuan.


5月12日 合入了pull request
【合入来源】
pytorch社区的test_nn.py中没有torch.nn.Module.npu的测试用例文件,故修改test/nn/test_nn_api.py这个测试用例文件,用于验证这个api的正确性
【修改方案】
用例设计原则
本次验证结合官方用例复用与自定义用例补充的方式,遵循 “最小化、核心化、针对性” 原则,核心思路为:
补充缺失用例:针对torch.nn.Module.npu这个缺失用例,我们设计了精简且详细的测试用例,覆盖了其核心功能
聚焦核心功能:我们验证了这个 API 的基础操作以及功能,确保结果反映 API 本身适配状态;
轻量化设计:单个测试方法仅验证一个核心功能点,逻辑清晰、执行高效,便于问题定位。
【资料变更】
不涉及
【接口变更】
不涉及
【功能验证】
在2.9.0 版本上执行该用例,测试通过,日志如下:
【CheckList】