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

Fix test layers unitest error #24946

Merged
merged 2 commits into from
Jun 6, 2020

Conversation

phlrain
Copy link
Collaborator

@phlrain phlrain commented Jun 5, 2020

PR types

Bug fixs

PR changes

Others

Describe

修复 test_layers 在cpu下的错误
test_layers 在cpu下,动态图和静态图结果不一致,分析发现是动态图中用到了 zero_copy的问题,设置zero_copy 等于 false

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jun 5, 2020

Thanks for your contribution!
Please add test = develop in your commit message to trigger CI to ensure your PR can be merged.
See Paddle CI Manual for details.

@paddle-bot-old
Copy link

paddle-bot-old bot commented Jun 5, 2020

❌This PR is not created using PR's template. You can refer to this Demo.
Please use PR's template, it helps save our maintainers' time so that more developers get helped.

@zhiqiu zhiqiu self-requested a review June 5, 2020 13:19
Copy link
Contributor

@zhiqiu zhiqiu left a comment

Choose a reason for hiding this comment

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

LGTM for fixing test_layers.
We should find out why it fails when zero_copy=True.

@phlrain phlrain merged commit c233328 into PaddlePaddle:develop Jun 6, 2020
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