Skip to content

Conversation

@ForFishes
Copy link
Member

PR types

Bug fixes

PR changes

Others

Describe

Others

@paddle-bot-old
Copy link

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

@ForFishes ForFishes changed the title Others fix safe bug of scatter/scatter_nd Jun 30, 2021
"The index is out of bounds, "
"please check whether the dimensions of index and "
"input meet the requirements. It should "
"be greater or equal to 0, but received [%d]",
Copy link
Contributor

Choose a reason for hiding this comment

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

greater than or equal to 0? 我英语也不太好,确认下,下面都是这样写的

Copy link
Member Author

Choose a reason for hiding this comment

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

额,有点问题,我修改一下.

Copy link
Contributor

@chenwhql chenwhql left a comment

Choose a reason for hiding this comment

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

LGTM

@ForFishes ForFishes merged commit c522530 into PaddlePaddle:develop Jul 1, 2021
@ForFishes ForFishes deleted the fix_safe_scattter branch July 1, 2021 09:55
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.

2 participants