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

【Hackathon No.62】增加pool3d算子BF16及单测,lgamma, masked_select FP16/BF16算子单测 #51837

Merged
merged 2 commits into from
Apr 4, 2023

Conversation

co63oc
Copy link
Contributor

@co63oc co63oc commented Mar 20, 2023

PR types

Others

PR changes

Others

Describe

增加pool3d算子BF16及单测

增加lgamma FP16/BF16算子单测
文档修改 PaddlePaddle/docs#5758

增加masked_select FP16/BF16算子单测
文档修改 PaddlePaddle/docs#5767

有的BF16需要传入place参数,如果不传place出现找不到kernel函数错误

@Vvsmile
Copy link
Contributor

Vvsmile commented Mar 20, 2023

请核查一下是否具有API Check任务

@co63oc co63oc force-pushed the test20 branch 3 times, most recently from c779537 to dd50307 Compare March 20, 2023 10:17
@co63oc
Copy link
Contributor Author

co63oc commented Mar 20, 2023

请核查一下是否具有API Check任务

已添加文件python/paddle/fluid/tests/unittests/test_pool3d_api.py 测试 test_static_bf16_gpu

@paddle-bot paddle-bot bot added the contributor External developers label Mar 20, 2023
@co63oc co63oc force-pushed the test20 branch 3 times, most recently from a45ca8b to d509f53 Compare March 23, 2023 12:57
@co63oc co63oc changed the title 【Hackathon No.62】增加pool3d算子BF16及单测 【Hackathon No.62】增加pool3d算子BF16及单测,lgamma FP16/BF16算子单测 Apr 1, 2023
@co63oc co63oc changed the title 【Hackathon No.62】增加pool3d算子BF16及单测,lgamma FP16/BF16算子单测 【Hackathon No.62】增加pool3d算子BF16及单测,lgamma, masked_select FP16/BF16算子单测 Apr 2, 2023
@Vvsmile
Copy link
Contributor

Vvsmile commented Apr 3, 2023

LGTM,但是需要解决一下CI问题

@co63oc
Copy link
Contributor Author

co63oc commented Apr 4, 2023

LGTM,但是需要解决一下CI问题

CI 已完成,PR-CI-Kunlun-KP-Build 是其他CI有同样问题
图片

luotao1
luotao1 previously approved these changes Apr 4, 2023
ZzSean
ZzSean previously approved these changes Apr 4, 2023
Copy link
Contributor

@ZzSean ZzSean left a comment

Choose a reason for hiding this comment

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

LGTM,一些小问题后续可以提个PR进行修改

place = core.CUDAPlace(0)
if core.is_bfloat16_supported(place):
if core.is_compiled_with_rocm():
self.check_output_with_place(place)
Copy link
Contributor

Choose a reason for hiding this comment

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

这两个分支完全一样,可以删掉

def test_check_output(self):
if core.is_compiled_with_cuda():
place = core.CUDAPlace(0)
if core.is_bfloat16_supported(place):
Copy link
Contributor

Choose a reason for hiding this comment

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

测例前有skipIf,所以if core.is_compiled_with_cuda()if core.is_bfloat16_supported(place)这两个判断可以去掉了

@luotao1
Copy link
Contributor

luotao1 commented Apr 4, 2023

@co63oc 这两个小问题比较容易改,直接在这个PR里修改吧

@co63oc co63oc dismissed stale reviews from ZzSean and luotao1 via e4e4e31 April 4, 2023 03:09
@co63oc
Copy link
Contributor Author

co63oc commented Apr 4, 2023

这两个小问题比较容易改,直接在这个PR里修改吧
@luotao1

已修改,CI已检查完成,PR-CI-Static-Check 是文档修改PR的检查
图片

Copy link
Contributor

@ZzSean ZzSean left a comment

Choose a reason for hiding this comment

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

LGTM

@luotao1 luotao1 merged commit b0dbf9f into PaddlePaddle:develop Apr 4, 2023
@co63oc co63oc deleted the test20 branch May 11, 2023 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants