Skip to content

add deprecated for gather api and refine the doc of gather api#26579

Merged
wangchaochaohu merged 6 commits intoPaddlePaddle:developfrom
wangchaochaohu:add_decorator
Aug 25, 2020
Merged

add deprecated for gather api and refine the doc of gather api#26579
wangchaochaohu merged 6 commits intoPaddlePaddle:developfrom
wangchaochaohu:add_decorator

Conversation

@wangchaochaohu
Copy link
Contributor

@wangchaochaohu wangchaochaohu commented Aug 23, 2020

PR types

Function optimization

PR changes

Docs

Describe

add deprecated for gather(gather_nd) api and refine the doc of gather(gather_nd) api

image

image
image

@paddle-bot-old
Copy link

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

This was referenced Aug 23, 2020
zhupengyang
zhupengyang previously approved these changes Aug 24, 2020
Copy link
Contributor

@zhupengyang zhupengyang left a comment

Choose a reason for hiding this comment

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

LGTM

TCChenlong
TCChenlong previously approved these changes Aug 24, 2020
Copy link
Contributor

@TCChenlong TCChenlong left a comment

Choose a reason for hiding this comment

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

LGTM

zhupengyang
zhupengyang previously approved these changes Aug 24, 2020
@wangchaochaohu wangchaochaohu dismissed stale reviews from zhupengyang and TCChenlong via 521a266 August 24, 2020 11:51
Copy link
Contributor

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

@XiaoguangHu01 XiaoguangHu01 left a comment

Choose a reason for hiding this comment

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

LGTM

@wangchaochaohu wangchaochaohu merged commit a31dbc8 into PaddlePaddle:develop Aug 25, 2020
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.

4 participants