Skip to content

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Nov 29, 2023

PR types

Others

PR changes

Others

Description

移除 test_resnet,将原来的 test_resnet_v2 rename 为 test_resnet,因为 test_resnet_v2 只是 #27459 引入的用 2.0 API 重写的 test_resnet 而已,现在 fluid 清理后 test_resnet 已经基本完全一样了,没必要维护两份同样的单测

另外只测一种情况,降低单测时间

PCard-66972

Copy link
Contributor

@XieYunshen XieYunshen left a comment

Choose a reason for hiding this comment

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

LGTM
单测删除

Copy link
Contributor

@Aurelius84 Aurelius84 left a comment

Choose a reason for hiding this comment

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

LGTM

@SigureMo SigureMo changed the title [Dy2St] Enable dy2st resnet tests [Dy2St] Remove duplicate dy2st resnet test Dec 5, 2023
@SigureMo SigureMo merged commit 9d7150b into PaddlePaddle:develop Dec 5, 2023
@SigureMo SigureMo deleted the dy2st/enable-test-resnet-uts branch December 5, 2023 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants