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

[Dy2St] Fix test_seq2seq can not switch SOT and AST mode #59058

Merged
merged 2 commits into from
Nov 17, 2023

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Nov 16, 2023

PR types

Bug fixes

PR changes

Others

Description

  • 修复 test_seq2seq 无法切换 SOT 和 AST 模式,导致全部跑在了 SOT 模式,而 SOT 模式 Cache 一直不命中,就非常慢,导致随机 TIMEOUT 的问题
  • 修复 _to_static_mode_guard_ 内报错时无法正常切换回非动转静模式,导致动态图走错分支的问题

同时摸底本单测的最终态问题,目前挂在 API paddle.nn.functional.sequence_mask 尚不支持,该任务为 #58067 202 题

PCard-66972

Copy link
Contributor

@2742195759 2742195759 left a comment

Choose a reason for hiding this comment

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

LGTM

@2742195759 2742195759 merged commit 50b5ec0 into PaddlePaddle:develop Nov 17, 2023
@2742195759 2742195759 deleted the dy2st/re-enable-seq2seq branch November 17, 2023 03:36
SecretXV pushed a commit to SecretXV/Paddle that referenced this pull request Nov 28, 2023
…dle#59058)

* [Dy2St] Fix `test_seq2seq` can not switch SOT and AST mode

* refine
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.

2 participants