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

No.55 complete argsort FP16 test, add argsort BF16 support and test #51823

Closed
wants to merge 52 commits into from

Conversation

superwinner1
Copy link
Contributor

@superwinner1 superwinner1 commented Mar 19, 2023

PR types

Others

PR changes

Others

Description

complete argsort FP16 test, add argsort BF16 support and test

@superwinner1 superwinner1 changed the title complete argsort FP16 test, add argsort BF16 support and test No.55complete argsort FP16 test, add argsort BF16 support and test Mar 19, 2023
@superwinner1 superwinner1 changed the title No.55complete argsort FP16 test, add argsort BF16 support and test No.55 complete argsort FP16 test, add argsort BF16 support and test Mar 19, 2023
@paddle-bot paddle-bot bot added the contributor External developers label Mar 19, 2023
@superwinner1
Copy link
Contributor Author

好的,我将按照您的建议进行进一步更改

@superwinner1
Copy link
Contributor Author

继承了OpTest类后,重写setUp()函数里面的self.op_type = "argsort"不对吗,CI测试结果提示我应该修改为self.class.op_type = "argsort"

or not core.is_bfloat16_supported(core.CUDAPlace(0)),
"core is not compiled with CUDA and not support the bfloat16",
)
class TestArgsortBF16OP(unittest.TestCase):
Copy link
Contributor

Choose a reason for hiding this comment

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

这个我记得之前实现的时候是OpTest,现在为啥又换成了TestCase,最好继承OpTest,如果有单测问题不好解决可以留言帮忙解决

@paddle-ci-bot
Copy link

paddle-ci-bot bot commented Jun 7, 2023

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

@zhangting2020
Copy link
Contributor

你好,这个PR已经几乎接近完成了,目前看上去有一些代码冲突,可以解决下冲突,如果没有其他问题应该就可以合入了

@luotao1
Copy link
Contributor

luotao1 commented Aug 9, 2023

close due to the following PR is merged:

@luotao1 luotao1 closed this Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants