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

【PIR API adaptor No.224】 Migrate paddle.tan #58737

Merged
merged 2 commits into from
Nov 8, 2023

Conversation

xiaoyewww
Copy link
Contributor

@xiaoyewww xiaoyewww commented Nov 6, 2023

PR types

Others

PR changes

APIs

Description

PIR API 推全升级
将 paddle.Tensor.log2 迁移升级至 pir,并更新单测

#58067

单测覆盖率:4/4

@paddle-bot paddle-bot bot added the contributor External developers label Nov 6, 2023
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 7, 2023
Copy link
Contributor

@MarioLulab MarioLulab left a comment

Choose a reason for hiding this comment

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

nice work ~ 但还有某些单测有所遗漏,需要添加上去

@@ -1960,7 +1960,7 @@ def init_shape(self):
def test_check_grad(self):
if self.dtype == np.float16:
return
self.check_grad(['X'], 'Out')
self.check_grad(['X'], 'Out', check_pir=True)
Copy link
Contributor

Choose a reason for hiding this comment

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

TestTan 继承自 TestActivation. TestActivation 有一个 test_check_output 需要继承重写并打开 check_pir 开关。麻烦添加一下吧~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@@ -4745,7 +4746,7 @@ def test_check_grad(self):
check_pir=True,
)
create_test_act_fp16_class(TestCos, check_pir=True)
create_test_act_fp16_class(TestTan)
create_test_act_fp16_class(TestTan, check_pir=True)
Copy link
Contributor

Choose a reason for hiding this comment

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

类似于 create_test_act_fp16_class, 在该文件内还有一个 create_test_act_bf16_class(TestTan) 创建了 bf16 的单测,麻烦也打开一下吧~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

@MarioLulab MarioLulab left a comment

Choose a reason for hiding this comment

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

LGTM

@YuanRisheng YuanRisheng merged commit 8022b63 into PaddlePaddle:develop Nov 8, 2023
28 checks passed
@xiaoyewww xiaoyewww deleted the new-ir/tan branch November 8, 2023 05:55
danleifeng pushed a commit to danleifeng/Paddle that referenced this pull request Nov 14, 2023
* feat(new-ir): support tan

* feat(new-ir): update ut of tan
SecretXV pushed a commit to SecretXV/Paddle that referenced this pull request Nov 28, 2023
* feat(new-ir): support tan

* feat(new-ir): update ut of tan
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants