Skip to content

[Auto Parallel] Add spmd rule No.18 for conv3d and conv3d_grad ops. #72882

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

Merged
merged 4 commits into from
Jul 4, 2025

Conversation

NKNaN
Copy link
Contributor

@NKNaN NKNaN commented May 22, 2025

PR Category

Auto Parallel

PR Types

New features

Description

为 conv3d 和 conv3d_grad 算子增加切分推导规则。

  • 规则与现有 conv2d 规则一致。

Copy link

paddle-bot bot commented May 22, 2025

你的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.

Copy link

paddle-ci-bot bot commented Jun 15, 2025

Sorry to inform you that 7d78a2d's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually.

filter_dims_mapping.size()));
// todo: NCDHW or NDHWC check, check channel logic, input's channel
// dims_mapping must be equal to filter's channel dims_mapping
int inpput_channel_dim = (data_format == "NCDHW") ? 1 : 4;
Copy link
Contributor

Choose a reason for hiding this comment

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

inpput_channel_dim -> input_channel_dim

Copy link
Contributor Author

Choose a reason for hiding this comment

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

revised

@luotao1
Copy link
Contributor

luotao1 commented Jun 23, 2025

如果Clone还是失败的话,请merge develop来解决

@NKNaN NKNaN force-pushed the conv3d-spmd branch 3 times, most recently from c2c1f30 to 262694d Compare June 23, 2025 13:03
@NKNaN
Copy link
Contributor Author

NKNaN commented Jul 2, 2025

/re-run all-failed

@luotao1 luotao1 merged commit 47ad81f into PaddlePaddle:develop Jul 4, 2025
79 of 89 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants