Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

【api 2.0】 reduce_all-->all reduce_any-->any #28181

Closed
wants to merge 5 commits into from
Closed

【api 2.0】 reduce_all-->all reduce_any-->any #28181

wants to merge 5 commits into from

Conversation

liuhuiCNN
Copy link
Contributor

@liuhuiCNN liuhuiCNN commented Oct 21, 2020

PR types

Function optimization

PR changes

APIs

Describe

merge #28134 and #28098 to this one PR
(1) reduce_all-->all
(2) reduce_any-->any
(3) update doc in reduce_all and reduce_any

@liuhuiCNN liuhuiCNN changed the title 2.0rc api rename1 【api 2.0】 reduce_all-->all reduce_any-->any Oct 21, 2020
cryoco and others added 3 commits October 21, 2020 19:57
add paddle.enable_static() on sample code

alias recude_all-->all, reduce_any-->any

add import reduce_all and reduce_any in python/paddle/tensor/math.py

import all and any in python/paddle/tensor/__init__.py

remove all and any OP in python/paddle/tensor/logic.py, add all and any OP in python/paddle/tensor/math.py

fix import error

remove TestAllAPI temporary
jzhang533
jzhang533 previously approved these changes Oct 21, 2020
Copy link
Contributor

@jzhang533 jzhang533 left a comment

Choose a reason for hiding this comment

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

lgtm

XiaoguangHu01
XiaoguangHu01 previously approved these changes Oct 21, 2020
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

Copy link
Contributor

@jzhang533 jzhang533 left a comment

Choose a reason for hiding this comment

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

lgtm

@liuhuiCNN
Copy link
Contributor Author

move to #28215

@liuhuiCNN liuhuiCNN closed this Oct 26, 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