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

refine pow module and its test #5319

Merged
merged 8 commits into from
Jun 30, 2021

Conversation

mosout
Copy link
Contributor

@mosout mosout commented Jun 27, 2021

doctest:

image

docstring:

image

@@ -1013,8 +1013,18 @@ def forward(self, x, y):
@experimental_api
def pow_op(tensor, exponent):
r"""Takes the power of each element in input with exponent and returns a tensor with the result.
exponent can be either a single float number or a single int number.

exponent can be either a single float number, a single int number, or a tensor with the same shape as input.
Copy link
Contributor

@doombeaker doombeaker Jun 29, 2021

Choose a reason for hiding this comment

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

需要在 PR 的 description 里附上 docstring 编译后的截图(顺便也检查了编译文档时有否warning,现在文档编译有warning也是不能通过CI测试的)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

加上了

Copy link
Contributor

Choose a reason for hiding this comment

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

呃……还是不符合要求。你可以参考下已经合并的PR,比如这个 #5101

Copy link
Contributor Author

Choose a reason for hiding this comment

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

好的

@mosout mosout requested a review from doombeaker June 30, 2021 05:47
@oneflow-ci-bot oneflow-ci-bot removed their request for review June 30, 2021 14:51
@oneflow-ci-bot oneflow-ci-bot self-requested a review June 30, 2021 14:51
@jackalcooper jackalcooper merged commit da69f6b into Oneflow-Inc:master Jun 30, 2021
@doombeaker doombeaker removed the request for review from oneflow-ci-bot July 1, 2021 15:19
@mosout mosout deleted the refine_pow_module branch December 14, 2021 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants