已合并
【社区任务】Ascend for PyTorch测试用例解耦(6) 交付件 #33
【社区任务】Ascend for PyTorch测试用例解耦(6) 交付件 #33
已合并
kevin_lee1231创建于 21 天前
kevin_lee1231
kevin_lee1231
21 天前

本次提交为【社区任务】Ascend for PyTorch 测试用例解耦(6) 交付件,目录按任务编号与改造测试文件命名,包含解耦交付报告、NPU 测试日志、运行截图,用于任务验收。

任务信息

目标测试文件:test/ao/sparsity/test_qlinear_packed_params.py
场景:场景四(无需解耦改造,NPU 测试失败)
涉及模块:torch.ao 稀疏动态量化线性层(sparsity)

核心结论

1、已完成 test_qlinear_packed_params.py 测试用例硬件分类标注:该文件为 CPU 量化后端测试,无 CUDA 强绑定守卫,本次仅新增 HardwareClassification.GENERIC 分类标签,测试方法、输入、断言与执行逻辑均未改动;改造前后用例集合不变,不影响原有 CPU/GPU 社区 CI。

2、PyTorch 上游社区 PR:https://github.com/pytorch/pytorch/pull/193177 ,状态:Merged(合入提交 4b5f5054e13d)。

3、TorchNPU 变更:无需修改。

本次仅修改测试用例标注,不引入新的设备接口,TorchNPU 侧无需变更,现有反序列化补丁保持原样。

4、NPU 验证:在 A2(910B3)和 A5(950PR)平台完成自测,共 3 个用例,通过 0,失败 3,跳过 0。默认运行与 GENERIC 筛选运行均在 torch.load 的 weights-only 反序列化路径报错,未修改源码在各自相同环境中出现相同错误;单独设置 TORCH_FORCE_NO_WEIGHTS_ONLY_LOAD=1 的诊断运行为 3/3 通过。失败用例已做根因归类为框架兼容性,非解耦代码问题,默认错误与诊断结果均如实保留。

交付文件清单

task_6_test_qlinear_packed_params/
├── test_qlinear_packed_params解耦交付报告.md
├── log/
│   ├── A2-test_qlinear_packed_params.log
│   └── A5-test_qlinear_packed_params.log
└── screenshots/
    ├── A2-test_qlinear_packed_params.png
    └── A5-test_qlinear_packed_params.png

交付报告第 5 节说明两份日志与两张终端截图的内容和用途。日志统一放在 log/,截图统一放在 screenshots/,通过文件名中的 A2、A5 区分平台。两份日志均为未设置诊断变量的完整 GENERIC 运行记录,原始日志和截图内容保持不变。分类统计中的 passed=3 表示选中 3 个测试,实际功能结果为通过 0、错误 3。

本次仅提交上述 5 个文件。其余分项日志、运行脚本、lint 记录和元数据先在本地留存。

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 kevin_lee1231 的贡献)
ascend-robotascend-robot成员
21 天前 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
21 天前 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-Ascend/pytorch-ecosystem ✅ xuyun15, qianxiyue (2/2) ✅ qianxiyue (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

kevin_lee1231, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
ascend-robot
ascend-robot成员
21 天前 评论:

CLA Signature Pass

kevin_lee1231, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
kevin_lee1231kevin_lee1231
21 天前 修改了pull request 的描述
此处折叠了13条事件消息 查看更多
kevin_lee1231kevin_lee1231
13 天前 修改了pull request 的描述
hid87721478成员
5 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
5 天前 添加了label:approved
xuyun15成员
5 天前 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
5 天前 添加了label:lgtm
ascend-robotascend-robot成员
5 天前 合入了pull request