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.160、204、216 】 Migrate NLLLoss/BCEWithLogitsLoss/MarginRankingLoss into pir #58832

Merged
merged 14 commits into from
Nov 27, 2023
Merged

Conversation

DrRyanHuang
Copy link
Member

@DrRyanHuang DrRyanHuang commented Nov 8, 2023

PR types

Others

PR changes

APIs

Description

create_test_case 函数中创建多个 MarginRankingLossCls 类, run_static_api 函数未添加 test_with_pir_api 装饰器,由于PIR模式下不支持 *.name

self.assertTrue('loss' in result.name)

2023-11-18 15:15:00 InvalidArgumentError: Currently, we can only get name of OpResult that is persistable (at ../paddle/fluid/pybind/pir.cc:743)
  • test_nll_loss, 单测通过率 6/7
    TestNLLLossName.test_name 未开启,PIR模式下不支持 *.name

  • test_sigmoid_cross_entropy_with_logits_op 单测通过率 8/9
    test/legacy_test/test_sigmoid_cross_entropy_with_logits_op.py 文件中 TestSigmoidCrossEntropyWithLogitsOpError.test_errors 未开启

Copy link

paddle-bot bot commented Nov 10, 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 Nov 10, 2023
@DrRyanHuang

This comment was marked as off-topic.

Comment on lines +1271 to +1273
if not in_pir_mode():
# TODO: Segmentation fault in pir mode
self.assertRaises(ValueError, test_x_shape_lt_1)
Copy link
Member Author

@DrRyanHuang DrRyanHuang Nov 25, 2023

Choose a reason for hiding this comment

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

PIR模式下会有 Segmentation fault, 所以加了if分支

@0x45f
Copy link
Contributor

0x45f commented Nov 27, 2023

麻烦在pr描述中写一下开启单测的比例

@0x45f 0x45f merged commit a53f409 into PaddlePaddle:develop Nov 27, 2023
@DrRyanHuang DrRyanHuang deleted the BCEWithLogitsLoss branch November 27, 2023 06:35
SecretXV pushed a commit to SecretXV/Paddle that referenced this pull request Nov 28, 2023
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.

2 participants