-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
tensor fluid code transfer part1 #41094
tensor fluid code transfer part1 #41094
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
@@ -14,7 +14,7 @@ | |||
|
|||
from ..fluid.layer_helper import LayerHelper |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from ..framework import LayerHelper?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK,在另一个还在修改的pr中修改。
python/paddle/tensor/math.py
Outdated
from ..fluid.layers import reduce_prod | ||
from ..fluid.layers import elementwise_sub |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to decouple with reduce_prod/elementwise_sub or move them outside of fluid?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reduce_prod已修改。
elementwise系列op与新替代API不完全兼容,应该要统一迁出,确定新路径后修改。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, remind Todos
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
||
out = b * \\frac{e^{a * x} - e^{-a * x}}{e^{a * x} + e^{-a * x}} | ||
|
||
Parameters: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
其他API用的是Args,这里用的是Parameters,要保持一致。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
好的,这个可能就旧代码风格,考虑这个已经approve,在另一个pr中修改。
PR types
Others
PR changes
APIs
Describe
tensor fluid code transfer part1