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.251-252】 paddle.vision.ops.yolo_box and yolo_loss #58782

Merged
merged 9 commits into from
Nov 15, 2023

Conversation

Liyulingyue
Copy link
Contributor

@Liyulingyue Liyulingyue commented Nov 7, 2023

PR types

Others

PR changes

APIs

Description

#58067

yolo_box:单测全
yolo_loss:继承自OpTest的单测忘记打开了

关于yolo_loss当前找到了test_yolov3_loss_op.py、test_device_guard.py两个测试文件,麻烦帮忙确认一下IR测试应当加到哪个文件里。

@Liyulingyue Liyulingyue changed the title 【PIR API adaptor No.252】 paddle.vision.ops.yolo_loss 【PIR API adaptor No.251-252】 paddle.vision.ops.yolo_box and yolo_loss Nov 7, 2023
@paddle-bot paddle-bot bot added the contributor External developers label Nov 7, 2023
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 8, 2023
@MarioLulab
Copy link
Contributor

关于 yolo_box : 在 test/legacy_test/test_yolo_box_op.py 文件下还遗漏了一个 TestYoloBoxStatic,可以使用 @test_with_pir_api 来为单测开启 pir 模式

关于 yolo_loss : test_yolov3_loss_op.py、test_device_guard.py 文件都可以添加 pir 测试

@Liyulingyue
Copy link
Contributor Author

关于 yolo_box : 在 test/legacy_test/test_yolo_box_op.py 文件下还遗漏了一个 TestYoloBoxStatic,可以使用 @test_with_pir_api 来为单测开启 pir 模式

关于 yolo_loss : test_yolov3_loss_op.py、test_device_guard.py 文件都可以添加 pir 测试

get~

@0x45f
Copy link
Contributor

0x45f commented Nov 14, 2023

test_device_guard单测先跳过吧,先只修改另外两个单测文件~

test/legacy_test/test_device_guard.py Outdated Show resolved Hide resolved
test/legacy_test/test_device_guard.py Outdated Show resolved Hide resolved
@luotao1 luotao1 merged commit fb7c80e into PaddlePaddle:develop Nov 15, 2023
28 checks passed
@Liyulingyue Liyulingyue deleted the IR252 branch January 15, 2024 14:20
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.

5 participants