已合并
迁移experimental的63/65/71三个样例至examples #1087
sunhao_hw创建于 8月11日
迁移experimental的63/65/71三个样例至examples #1087
已合并
sunhao_hw创建于 8月11日
sunhao_hw成员
8月11日

描述

涉及样例:

  • 63_ascend950_dual_level_quant_mx_batch_matmul
  • 65_ascend950_fp8_mx_grouped_matmul_slice_m_swiglu_mx_quant
  • 71_ascend950_fp8_mx_grouped_matmul_finalize_routing

optest/atk测试交付件均完备

关联的Issue

原因

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 sunhao_hw 的贡献)
Ssunhao_hw成员
8月11日 创建了 pull request,commit 43fac478
atomgit-bot
atomgit-bot
8月11日 评论:

变更摘要

此 PR 将三个原本位于 experimental 目录下的 Ascend950 相关样例(编号 63、65、71)迁移至 examples/ 目录,统一目录命名格式并修正示例代码中的路径引用;同时新增对应的 optest 自动化测试脚本,确保迁移后功能完备可验证。

主要改动

  • 样例目录重命名与迁移:将 ascend950_dual_level_quant_mx_batch_matmul、ascend950_fp8_mx_grouped_matmul_slice_m_swiglu_mx_quant、ascend950_fp8_mx_grouped_matmul_finalize_routing 三个样例目录分别重命名为带编号前缀的 63_、65_、71_ 形式并移入 examples/,目录内的 gen_data.py、compare.py 等脚本随目录同步迁移。
  • C++ 示例中路径常量更新:在 dual_level_quant_mx_batch_matmul.cpp 中将 kDataRoot 从 "examples/ascend950_dual_level_quant_mx_batch_matmul/data" 更新为 "examples/63_ascend950_dual_level_quant_mx_batch_matmul/data";类似地,在 fp8_mx_grouped_matmul_slice_m_swiglu_mx_quant.cpp 中将 kExampleRoot 更新为带 65_ 前缀的新路径,确保编译后的可执行文件能正确定位数据目录。
  • Python 辅助脚本中算子名修正:在 gen_data_compare.py 中将 op_name 从 "ascend950_fp8_mx_grouped_matmul_finalize_routing" 改为 "71_ascend950_fp8_mx_grouped_matmul_finalize_routing",以保证生成的可执行文件路径与 CMake 构建输出一致。
  • 新增 optest 测试用例:为三个样例分别新增 test_63_ascend950_dual_level_quant_mx_batch_matmul.py、test_65_ascend950_fp8_mx_grouped_matmul_slice_m_swiglu_mx_quant.py 和 test_71_ascend950_fp8_mx_grouped_matmul_finalize_routing.py,通过 pytest.mark.skipif 限定仅在 Ascend950 设备上执行,覆盖了输出 shape/dtype/device 校验以及与 golden 参考值的精度/字节匹配对比。
likedislike
不准确?
atomgit-bot
atomgit-bot
8月11日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
8月11日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
8月11日 评论:

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
repo-cann/catlass ✅ yuantao_, longjihui (2/2) ✅ yuantao_ (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了65条消息 查看更多
longjihui
longjihui成员
8月13日 评论:

/lgtm

likedislike
yuantao
yuantao成员
8月13日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
8月13日 添加了label:lgtmapproved
CANN-robotCANN-robot成员
8月13日 合入了pull request
CANN-robot
CANN-robot成员
8月13日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike