Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Add Op UnitTest for batchnorm #1503

Merged
merged 22 commits into from
Jun 25, 2023
Merged

Conversation

Liyulingyue
Copy link
Contributor

@Liyulingyue Liyulingyue commented Jun 4, 2023

描述

From #1378

为Reduce类型算子增加新的测试用例

序号 算子名 单测文件
2 batch_norm test_batch_norm_op.py
3 batch_norm_grad test_batch_norm_op.py

算子类型

  • ElementWise:输入张量索引和输出张量索引之间存在一对一的对应关系
  • Broadcast:输入张量索引和输出张量索引之间存在一对多的对应关系
  • Injective:单射算子,可以将一个输出 axis 映射到一个输入 axis
  • Reduction:输入张量索引和输出张量索引之间存在多对一的对应关系
  • OutFusible:复杂算子,仍然可以将一对一的算子融合到其输出中。
  • kNonFusible:无法融合的算子

OpMapper

  • 该算子是否 OpMapper? 如果是,请贴出在 Paddle 中对应的 OpMaker 代码路径。(给出 Github 链接更好)

Test Cases Checklist

张量维度

  • 1D 张量
  • 2D 张量
  • 3D 张量
  • 4D 张量

special shape

挑选 2D/3D/4D 张量中的一个,测试下面的特殊情况。

  • 其中一个维度为 1
  • 其中一个维度小于 1024
  • 其中一个维度大于 1024
  • 向量的所有维度都是 1

张量数据类型

  • int32
  • int64
  • float16
  • float32
  • float64

广播

  • 这个算子是否支持广播?
  • 广播的测试样例

算子属性

算子属性的测试用例。

  • 属性:属性类型-可取值
    • op_type
    • axis
    • keepdim
  • 使用 OpTestHelper 测试上述属性的笛卡尔积组合

备注

@paddle-bot
Copy link

paddle-bot bot commented Jun 4, 2023

Thanks for your contribution!

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Jun 6, 2023
@Liyulingyue
Copy link
Contributor Author

@thisjiang @FisherWY

Copy link
Collaborator

@thisjiang thisjiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

文件有conflict了~另外麻烦用TestCaseHelper吧

@Liyulingyue
Copy link
Contributor Author

@thisjiang @FisherWY

@FisherWY
Copy link
Contributor

CINN目前已升级至C++17(PR1526),CI的Docker镜像也已同步更新,可以merge一下develop分支,防止出现CI失败的问题(cudnn.so未找到的问题)

git fetch origin
git merge origin/develop

@Liyulingyue
Copy link
Contributor Author

@thisjiang @FisherWY

Copy link
Collaborator

@thisjiang thisjiang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@thisjiang thisjiang merged commit b9fd519 into PaddlePaddle:develop Jun 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants