Skip to content

Conversation

@Shixiaowei02
Copy link
Contributor

@Shixiaowei02 Shixiaowei02 commented Jan 11, 2021

PR types

Bug fixes

PR changes

OPs

Describe

Paddle 2.0 前向算子版本升级信息补全(第二轮)的说明

Paddle 2.0 对部分算子进行了升级。为方便开发者明确算子版本,避免推理算子定义不一致导致计算错误,Paddle 制定了前向算子升级注册规范。目前仓储中仍有部分算子在 1.7 之后进行了升级但未注册,此提交合入后,注册信息将在 2.0 正式版得以补全。

注册信息的第二次补全增加了非兼容情况下的升级信息,以及 1.8 版本之后的兼容升级信息注册。补全工作是在 develop 分支上开展的全员任务,此提交为 release/2.0 的 cherry-pick。

此提交是独立的,不对算子正确性和框架主体功能带来风险。


附件一:前向算子版本升级检查点类型
image

附件二:前向算子升级补齐清单
截屏2021-01-1114 39 39
image

LielinJiang and others added 22 commits January 11, 2021 06:01
…addlePaddle#29923)

* add op version for fake_quant and fake_dequant ops, test=op_version, test=develop
* add gru op_register_version; test=op_version;

* Update fc,mul version;test=op_version;
change the elementwise ops version check, test=op_version
…dlePaddle#30011)

* add the support the op version check for matmul, test=op_version
* Register op version for linspace,test=op_version

* Register op version for linspace,test=op_version

* Register op version for linspace,test=op_version

* Register op version for linspace,test=op_version

* Register op version for linspace,test=op_version
…addlePaddle#30000)

* add trace op_register_version and fix defaulf bug; test=op_version

* add trace op_register_version; test=op_version

* add trace op_register_version; test=op_version

* add trace op_register_version; test=op_version

* fix missing the template bug of vector; test=op_version
@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

Superjomn
Superjomn previously approved these changes Jan 11, 2021
Copy link
Contributor

@Superjomn Superjomn left a comment

Choose a reason for hiding this comment

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

LGTM

@Shixiaowei02
Copy link
Contributor Author

Shixiaowei02 commented Jan 12, 2021

#29890 引入版本升级注册冲突,合入最新 release/2.0 分支解决。

Copy link
Contributor

@Superjomn Superjomn left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@raindrops2sea raindrops2sea left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Superjomn Superjomn left a comment

Choose a reason for hiding this comment

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

LGTM

@Superjomn Superjomn merged commit 5eab1a3 into PaddlePaddle:release/2.0 Jan 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.