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

【Error Message No. 4】 paddle/fluid/pir/transforms/* #62814

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

enkilee
Copy link
Contributor

@enkilee enkilee commented Mar 18, 2024

PR types

Others

PR changes

Others

Description

Copy link

paddle-bot bot commented Mar 18, 2024

你的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 the contributor External developers label Mar 18, 2024
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Mar 18, 2024
@Galaxy1458 Galaxy1458 merged commit 7986f42 into PaddlePaddle:develop Mar 19, 2024
30 checks passed
Comment on lines +46 to +50
PADDLE_ENFORCE_EQ(
add_input && conv2d_out,
true,
phi::errors::PreconditionNotMet("The type of add input should be the "
"same as the type of conv2d's out."));
Copy link
Contributor

Choose a reason for hiding this comment

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

这里的本意是add_input和conv2d_out相等,你这里是全为true。。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

收到

Comment on lines +248 to +251
PADDLE_ENFORCE_EQ(module_op.name(),
"builtin.module",
phi::errors::InvalidArgument(
"ShapeOptimizationPass should run on module op."));
Copy link
Contributor

Choose a reason for hiding this comment

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

这里的本意是module_op这个对象不为NULL,你这里是它的name等于"builtin.module"。。。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

收到

Comment on lines +49 to +52
PADDLE_ENFORCE_EQ(module_op.name(),
"builtin.module",
phi::errors::InvalidArgument(
"sub_graph_extract_pass should run on module op."));
Copy link
Contributor

Choose a reason for hiding this comment

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

同上

Copy link
Contributor Author

Choose a reason for hiding this comment

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

收到

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants