This repository has been archived by the owner on Jan 24, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 115
CINN 算子单测补充-ISSUES #1378
Comments
认领:2 3 |
认领:106、107 |
27 tasks
|
认领 scatter 相关的算子:65 66 123 124
|
认领 87, 88,89 |
认领 39,40 |
为了避免 CI 的压力,开发多几个算子单测再合并提交 PR。下面的状态为 Pre-PR 表示已经开发完毕,但是还没有提交 PR。 合并提交PR:#1401
|
认领 13, 14, 15 |
认领 126,127 |
认领 72, 73, 74, 75, 76,为了避免 CI 的测试压力,开发多个单元测试再合并提交一个 PR。对于部分存在问题的算子,需要进行修复,单独提交 PR。
|
21 tasks
认领以下算子的单测:
|
认领以下算子:
|
认领以下算子:
|
认领以下算子:
|
认领以下算子:
|
24 tasks
This comment was marked as duplicate.
This comment was marked as duplicate.
This was referenced Jun 4, 2023
认领一下算子:
|
|
CINN 算子单测补充基本都完成,感谢参与的小伙伴们!
欢迎继续参与快乐开源的其他任务! |
github-project-automation
bot
moved this from In Progress
to Done
in Call for Contributions
Jul 3, 2023
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
任务描述
大家好,近期我们在从dtype覆盖,shape覆盖,属性覆盖三个方面梳理CINN中的算子单测,并在此过程中修复单测中发现的BUG。我们一共统计了135个CINN算子,发现其中所有的算子单测都或多或少的缺失了一些测试用例,导致没能测试到某个dtype、shape或者属性,甚至有部分算子没有对应的测试文件。为保证和保障CINN算子的正确性,我们希望能尽可能补充完全CINN的算子单测用例。
注意事项
shape + shape相关attr
、dtype + dtype相关attr
、其它attr
将单测拆分为至少三个test class,不要全放在一个test class里任务要求
我们希望大家能从如下角度完善CINN中的算子单测:
[1024], [512, 256], [128, 64, 32], [16, 8, 4, 2], [16, 8, 4, 2, 1]
,以及特殊shape如[1]
,以及对该算子有特殊含义的shape如reduce算子的[1, 1, 1, 1, 1]
float32,float64,int8,int16,int32,int64,uint8
True/False
都应该测试到,及特殊取值,如scale算子的scale
属性特殊值0和1如若出现报错,请尽可能修复该问题。如若难度很大,请注释该配置,并在注释中加上TODO,同时在PR中加个comment,指明报错的配置和报错内容。
当前只需关注CUDA单测,跳过x86。
任务示例
如若之前从未尝试过编译和开发CINN,可通过如下脚本编译CINN:
CINN中算子单测位于python/tests/ops目录下,请仿照test_add_op_new.py或test_scale_op.py,利用
TestCaseHelper
来完善算子单测。可以直接使用
python
命令执行单测脚本,只需将CINN的build/python
目录加入到PYTHONPATH
环境变量中即可:可以通过
--case=TEST_CLASS1.TEST_CASE1,TEST_CLASS2.TEST_CASE2
这种方式来单独运行某个,或某几个case。当测试通过后,可按照代码贡献流程来向CINN贡献代码,建议一个PR只提交一个算子的单测及其修复代码。提交commit前请一定要先执行
pre-commit
。示例PR:#1379
为方便review,PR描述请参考示例PR(可直接拷贝)。
待完善算子单测列表 (整体进度:135/135)
addsubtractdividemultiplyfloor_dividemodmaxnormrelusigmoidceilsignclipThe text was updated successfully, but these errors were encountered: