Skip to content

[IPU] fix ut timeout 2#45233

Merged
qili93 merged 2 commits intoPaddlePaddle:developfrom
graphcore:ipu_commit/fix_ut_timeout_2
Sep 23, 2022
Merged

[IPU] fix ut timeout 2#45233
qili93 merged 2 commits intoPaddlePaddle:developfrom
graphcore:ipu_commit/fix_ut_timeout_2

Conversation

@gglin001
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

releated PR #45181
fix timeout for IPU UTs when using a physical IPU

@paddle-bot
Copy link

paddle-bot bot commented Aug 18, 2022

你的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 contributor External developers status: proposed labels Aug 18, 2022

# TODO(ipu) remove in the future version of popart
# keep the log clean, no side effects for tests without profiling
ipu_strategy.set_options(
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Contributor

@qili93 qili93 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

@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
单测时间设置

@gglin001
Copy link
Contributor Author

@qili93 hi, 如果没有更多的问题, 麻烦帮忙合并下这个PR, 感谢

Copy link
Contributor

@qili93 qili93 left a comment

Choose a reason for hiding this comment

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

LGTM

@qili93 qili93 merged commit 4dd5bf7 into PaddlePaddle:develop Sep 23, 2022
@gglin001 gglin001 deleted the ipu_commit/fix_ut_timeout_2 branch September 23, 2022 03:17
umiswing pushed a commit to umiswing/Paddle that referenced this pull request Sep 28, 2022
gglin001 added a commit to graphcore/Paddle-fork that referenced this pull request Oct 10, 2022
qingqing01 pushed a commit that referenced this pull request Oct 17, 2022
@paddle-bot-old paddle-bot-old bot removed the contributor External developers label Oct 17, 2022
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