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

complex No.15 sqrt op #57661

Closed
wants to merge 7 commits into from
Closed

Conversation

yangguohao
Copy link
Contributor

@yangguohao yangguohao commented Sep 23, 2023

PR types

Others

PR changes

OPs

Description

complex support for sqrt op
#56145

@paddle-bot
Copy link

paddle-bot bot commented Sep 23, 2023

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Sep 23, 2023
@yangguohao yangguohao force-pushed the complex_sqrt branch 2 times, most recently from 40f272f to 49be9db Compare September 23, 2023 08:12
@yangguohao
Copy link
Contributor Author

@luotao1 CI-Coverage 能不能帮我 rerun 一下

@luotao1 luotao1 added HappyOpenSource 快乐开源活动issue与PR HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 and removed HappyOpenSource 快乐开源活动issue与PR labels Sep 25, 2023
@luotao1 luotao1 changed the title complex sqrt op complex No.15 sqrt op Sep 25, 2023
@luotao1
Copy link
Contributor

luotao1 commented Sep 25, 2023

CI-Coverage 能不能帮我 rerun 一下

image

是覆盖率不够

@yangguohao
Copy link
Contributor Author

CI-Coverage 能不能帮我 rerun 一下

image 是覆盖率不够

这里的覆盖率是指代码没有运行到这个部分吗?这个应该如何解决


class TestSqrtComplex128(TestSqrt):
def init_dtype(self):
self.dtype = np.complex128
Copy link
Contributor

Choose a reason for hiding this comment

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

看下这两个单测是否跑进去了,如果跑进去的话,贴一个截图上来

@paddle-ci-bot
Copy link

paddle-ci-bot bot commented Oct 6, 2023

Sorry to inform you that 07d7182's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@paddle-ci-bot
Copy link

paddle-ci-bot bot commented Oct 31, 2023

Sorry to inform you that 53fbc3d's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@yangguohao
Copy link
Contributor Author

yangguohao commented Nov 9, 2023

看下这两个单测是否跑进去了,如果跑进去的话,贴一个截图上来

@luotao1
image
涛姐这样可以吗?这个还是 CI-Coverage 没覆盖

@@ -1732,7 +1732,9 @@ def _dfs_grad_op(op_desc, fwd_op_desc=None):
has_infer_inplace = base.core.has_infer_inplace(op_desc.type())
has_grad_op_maker = base.core.has_grad_op_maker(op_desc.type())
has_infer_inplace_in_grad_descendants = False
if not has_grad_op_maker:
# the OP test doesn't support higher order grad
is_grad_op_desc = op_desc.type().endswith('_grad')
Copy link
Contributor

Choose a reason for hiding this comment

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

为什么要修改这里的 op test呢

Copy link
Contributor Author

Choose a reason for hiding this comment

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

因为 OpTest 里应该是没有高阶导的测试,而这里的判断就避免 dfs 函数搜到高阶 grad op_desc。

Copy link

paddle-ci-bot bot commented Nov 16, 2023

Sorry to inform you that 2e9afec's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

Copy link

paddle-ci-bot bot commented Nov 26, 2023

Sorry to inform you that 236fdba's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

@yangguohao yangguohao closed this Dec 6, 2023
@yangguohao yangguohao deleted the complex_sqrt branch December 6, 2023 02:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants