Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Dy2St] lower time > 100 in dy2st unittests #59506

Merged
merged 6 commits into from
Dec 2, 2023

Conversation

ooooo-create
Copy link
Contributor

PR types

Others

PR changes

Others

Description

当前 #59339 记录的 time2 大于100esc的

test in dygraph_to_static 是否有单测 1x1(sec) time2(sec) 倍数
test_se_resnet #59312 347.43 652.07 1.88
test_tsm #59312 105.61 495.14 4.69
test_yolov3 #59312 106.65 475.92 4.46
test_seq2seq 175.53 242.77 1.38
test_build_strategy 65.57 106.53 1.62
test_basic_api_transformation #59312 30.12 104.18 3.46

其中 4a65cee 优化了四个,这里尝试优化其他两个,并可能更新 #59339 中表格数据

@SigureMo 我需要喵喵~

@paddle-bot paddle-bot bot added the contributor External developers label Nov 29, 2023
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 30, 2023
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

这个 PR 怎么变空了呢?

@ooooo-create
Copy link
Contributor Author

这个 PR 怎么变空了呢?
刚刚才发现有个 default and pir 的装饰器然后还原了一下,没来得及改 😂,回来提一下~

@SigureMo SigureMo changed the title [Dy2St] lower time > 100 in 单测 dygraph_to_static [Dy2St] lower time > 100 in dy2st unittests Dec 1, 2023
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Dec 1, 2023
@PaddlePaddle PaddlePaddle unlocked this conversation Dec 1, 2023
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

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

LGTMeow

@SigureMo SigureMo merged commit fed2230 into PaddlePaddle:develop Dec 2, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants