Skip to content

Conversation

@Charles-hit
Copy link
Contributor

@Charles-hit Charles-hit commented Sep 8, 2022

PR types

Others

PR changes

Others

Describe

目前assign二三阶yaml都是复用前向kernel,由于assign反向与前向实现一致,本次pr删除assign反向二三阶yaml,将assign反向一阶yaml改成invoke复用前向动态图实现无限高阶并且添加相应单测进行测试。需要注意的是,本次改动前反向都是inplace版本,但是前向没有inplace版本,因此复用的时候不再采用inplace版本。

@paddle-bot
Copy link

paddle-bot bot commented Sep 8, 2022

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@veyron95 veyron95 requested a review from zyfncg September 9, 2022 02:36
Copy link
Contributor

@veyron95 veyron95 left a comment

Choose a reason for hiding this comment

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

LGTM

@zyfncg zyfncg merged commit 65dd828 into PaddlePaddle:develop Sep 14, 2022
@Charles-hit Charles-hit deleted the refuse_foward_assign branch September 14, 2022 02:58
chenwhql pushed a commit to chenwhql/Paddle that referenced this pull request Sep 14, 2022
phlrain pushed a commit that referenced this pull request Sep 16, 2022
* normalize yaml file name (#45894) * Clear extra attributes of activation op in OpMaker (#45772) * clear extra attr of activation op in opmaker * fix syntax bug * fix mkldnn kernel * fix merge conflict * fix bug * [PHI] Normalize yaml op label (#45976) * normalize yaml op label * revert op_compat yaml change * fix prelu and rnn compat problem * replace api by op * support assign op backward refuse forward (#45879) * normize yaml backward op label (#46028) Co-authored-by: zyfncg <zhangyunfei07@baidu.com> Co-authored-by: Charles-hit <56987902+Charles-hit@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants