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

[PASS][HuaweiAscendNPU] Fix constant fold pass and solve the precision of ascend meshgrid op #9940

Merged

Conversation

shentanyue
Copy link
Collaborator

PR devices

Framework and NNadapter

PR types

Bug fixes

PR changes

PASS and OP

Description

修复常量折叠 Pass 和解决 Ascend meshgrid op 精度问题。

@paddle-bot
Copy link

paddle-bot bot commented Jan 18, 2023

Thanks for your contribution!

@shentanyue shentanyue force-pushed the fix_constant_fold_pass_and_meshgrid branch from f4cd74c to 39a103a Compare January 18, 2023 07:06
@PaddlePaddle PaddlePaddle locked as off-topic and limited conversation to collaborators Jan 18, 2023
@PaddlePaddle PaddlePaddle unlocked this conversation Jan 18, 2023
@@ -188,7 +188,6 @@ std::unique_ptr<RuntimeProgram> RunDefaultOptimizer(
"lite_conv_scale_fuse_pass",
"lite_conv_elementwise_tree_fuse_pass",
"lite_greater_than_cast_fuse_pass",
"fill_range_fuse_pass",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

已经有fill_constant和range的常量折叠pass了,这个pass不再需要

@@ -72,28 +92,26 @@ void RangeCalcOfflinePass::RemoveRangePattern(
auto step_t = step_var->GetMutable<lite::Tensor>();
if (!start_t->persistable() || !end_t->persistable() ||
!step_t->persistable()) {
LOG(WARNING) << "RangeCalcOfflinePass does not support input that is not "
"persistable";
VLOG(5)
Copy link
Collaborator

Choose a reason for hiding this comment

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

为什么把warning改成vlog

Copy link
Collaborator

@zhupengyang zhupengyang Jan 18, 2023

Choose a reason for hiding this comment

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

应该是不希望每次执行都打印太多log吧

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

是的,LOG(WARNING)会不受限制的进行打印,有时会输出大量log

Copy link
Collaborator

@zhupengyang zhupengyang left a comment

Choose a reason for hiding this comment

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

LGTM

@shentanyue shentanyue merged commit 5ccef53 into PaddlePaddle:develop Jan 19, 2023
@shentanyue shentanyue deleted the fix_constant_fold_pass_and_meshgrid branch January 19, 2023 02:47
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