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

[Phi] add the infer shape meta for the graph_send_recv #40320

Merged
merged 3 commits into from
Mar 10, 2022

Conversation

wawltor
Copy link
Contributor

@wawltor wawltor commented Mar 8, 2022

PR types

Performance optimization

PR changes

OPs

Describe

add the infer shape meta for the graph_send_recv

@paddle-bot-old
Copy link

paddle-bot-old bot commented Mar 8, 2022

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@wawltor wawltor changed the title add the infer shape meta for the graph_send_recv [Phi] add the infer shape meta for the graph_send_recv Mar 8, 2022
@@ -75,4 +75,10 @@ void AdadeltaInferMeta(const MetaTensor& param,
MetaTensor* avg_squared_grad_out,
MetaTensor* avg_squared_update_out);

void GraphSendRecvInferMeta(const MetaTensor& x,
Copy link
Contributor

Choose a reason for hiding this comment

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

三个输入的infermeta 函数放到 ternary.h 文件中。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

三个输入的infermeta 函数放到 ternary.h 文件中。

done

"Src_index and Dst_index should have the same shape."));

auto dims = x.dims();
out->set_dims(dims);
Copy link
Contributor

Choose a reason for hiding this comment

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

out 也需要设置一下dtype

Copy link
Contributor Author

Choose a reason for hiding this comment

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

out 也需要设置一下dtype

done

out->set_dims(dims);

if (pool_type == "MEAN") {
dst_count->set_dims({dims[0]});
Copy link
Contributor

Choose a reason for hiding this comment

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

dst_count 也需要设置一下dtype

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dst_count 也需要设置一下dtype

done

Copy link
Contributor

@MingMingShangTian MingMingShangTian left a comment

Choose a reason for hiding this comment

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

LGTM

@wawltor wawltor merged commit 5ae8513 into PaddlePaddle:develop Mar 10, 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.

2 participants