[Bug-Report|缺陷反馈]: 测试代码中 bias 形状定义错误-ops-nn-AtomGit

已关闭

[Bug-Report|缺陷反馈]: 测试代码中 bias 形状定义错误 #2478

旋风地虎创建于 5月3日关闭于 5月7日

旋风地虎

5月3日创建

Thanks for sending an issue! Please fill in the following template to help quickly solve your problem.

Describe the current behavior / 问题描述 (Mandatory / 必填)

在 ops-nn/matmul/quant_matmul/tests/ut/op_host/test_aclnn_quant_matmul.cpp:39 文件的第一个测试用例中：

TEST_F(l2_quant_matmul_test, ascend910B2_test_normal_input){
    auto x1_desc = TensorDesc({16, 32}, ACL_INT8, ACL_FORMAT_ND);
    auto x2_desc = TensorDesc({32, 16}, ACL_INT8, ACL_FORMAT_ND);
    auto bias_desc = TensorDesc({16, 16}, ACL_INT32, ACL_FORMAT_ND);  // 疑似存在错误！
    float deqScale = 1.0;
    auto out_desc = TensorDesc({16}, ACL_FLOAT16, ACL_FORMAT_ND);     //疑似存在错误
    ……
    // ut.TestPrecision();
}

Environment / 环境信息 (Mandatory / 必填)

昇腾，1*NPU 910C 40vCPUs 240GiB

Steps to reproduce the issue / 重现步骤 (Mandatory / 必填)

根据算子文档和代码中的校验逻辑：

bias 形状错误：在 aclnn_quant_matmul.cpp:251-260 中有明确的 bias 校验：

// 检查bias是否与n一致
if (bias != nullptr) { 
	op::Shape biasShape = bias->GetViewShape();
    int64_t biasNDim = biasShape[0]; // 0: 取bias的N
    OP_CHECK(biasNDim == x2NDim, ...);
}

对于 x2 = {32, 16}，N=16，所以 bias 应该是 {16} 而不是 {16, 16}
缺少 EXPECT 断言：即使定义了错误的参数，大部分测试用例的 EXPECT_EQ 都被注释掉了，无法捕获这类错误。

而作为对比，在同一文件的 V2 接口测试中（第79行），bias 定义是正确的
auto bias_desc = TensorDesc({3}, ACL_INT32, ACL_FORMAT_ND);