Skip to content

Conversation

@Shixiaowei02
Copy link
Contributor

@Shixiaowei02 Shixiaowei02 commented Dec 29, 2020

PR types

Bug fixes

PR changes

Others

Describe

解决 REGISTER_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.

@Shixiaowei02 Shixiaowei02 force-pushed the dev/op_attr_wrapper branch 2 times, most recently from 191c867 to 1280a41 Compare December 29, 2020 12:52
@Shixiaowei02 Shixiaowei02 changed the title fix a bug in op_version_registry, test=develop fix a bug in op_version_registry, test=develop, test=op_version Dec 29, 2020
Copy link
Contributor

@Channingss Channingss left a comment

Choose a reason for hiding this comment

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

LGTM

Channingss
Channingss previously approved these changes Dec 30, 2020
Copy link
Contributor

@Channingss Channingss left a comment

Choose a reason for hiding this comment

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

LGTM

DannyIsFunny
DannyIsFunny previously approved these changes Dec 30, 2020
Copy link
Contributor

@DannyIsFunny DannyIsFunny 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

@DannyIsFunny DannyIsFunny 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 Shixiaowei02 merged commit 53bb126 into PaddlePaddle:develop Jan 6, 2021
Superjomn pushed a commit that referenced this pull request Jan 13, 2021
…ase/2.0 (#30292)

* Register op version for grid_sampler, test=op_version (#29916)

* add op version for fake_quant and fake_dequant ops, test=op_version (#29923)

* Register op version for print, test=op_version (#29945)

* add gru op_register_version; test=op_version; (#29931)

* Register op version for coalesce_tensor. (#29940)

* register op version for conv2d_transpose, conv3d_transpose and depthwise_conv2d_transpose, test=op_version (#29937)

* add op_register_version for allclose op; test=op_version (#29968)

* register ModifyAttr for instance_norm, test=op_version (#29938)

* add op_version for flip op [test=op_version] (#30019)

* add the op version check for the elementwise ops, test=op_version (#30010)

* add the support the op version check for matmul, test=op_version (#30011)

* Revert "register ModifyAttr for instance_norm, test=op_version (#29938)"

* add REGISTER_OP_VERSION for generate_proposals, roi_align, roi_pool test=op_version (#30034)

* Fix rank_attention op_version, test=op_version (#30006)

* fix rank_attention, test=op_version

* Register op version for linspace,test=op_version (#30025)

* fix op_register_version for compare ops, test=op_version (#30007)

Co-authored-by: zhoushunjie <zhoushunjie@baidu.com>

* register ModifyAttr for instance_norm, test=op_version (#30065)

* register instance norm, test=op_version

* add trace op_register_version and fix version bug; test=op_version (#30000)

* fix a bug in op_version_registry, test=develop, test=op_version (#29994)

* Add version checking, test=op_version (#30129)

* fix a bug in gaussian_random_op version, test=release/2.0

Co-authored-by: LielinJiang <50691816+LielinJiang@users.noreply.github.com>
Co-authored-by: cc <52520497+juncaipeng@users.noreply.github.com>
Co-authored-by: Qi Li <qili93@qq.com>
Co-authored-by: Jack Zhou <zhoushunjie@baidu.com>
Co-authored-by: Guo Sheng <whucsgs@163.com>
Co-authored-by: wangxinxin08 <69842442+wangxinxin08@users.noreply.github.com>
Co-authored-by: wawltor <fangzeyang0904@hotmail.com>
Co-authored-by: FlyingQianMM <245467267@qq.com>
Co-authored-by: ceci3 <ceci3@users.noreply.github.com>
Co-authored-by: hutuxian <hutuxian2011@sina.cn>
Co-authored-by: chalsliu <45041955+chalsliu@users.noreply.github.com>
Co-authored-by: wangguanzhong <jerrywgz@126.com>
Co-authored-by: ShenLiang <shenliang03@baidu.com>
Co-authored-by: yinhaofeng <66763551+yinhaofeng@users.noreply.github.com>
Co-authored-by: channings <chenlingchi@baidu.com>
Co-authored-by: chentianyu03 <chentianyu03@baidu.com>
Co-authored-by: ruri <shipeng1108@163.com>
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.

3 participants