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

【SCU】【PPSCI Export&Infer No.10】 cfdgcn #1037

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

yangrongxinuser
Copy link

PR types

Others

PR changes

Others

Describe

为cfdgcn算子添加export和inference功能

Copy link

paddle-bot bot commented Dec 5, 2024

Thanks for your contribution!

Copy link
Collaborator

Choose a reason for hiding this comment

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

这个pre-commit应该不用改吧

@@ -53,4 +53,4 @@ repos:
files: \.(c|cc|cxx|cpp|cu|h|hpp|hxx|cuh|proto)$

exclude: |
^jointContribution/
^jointContribution/
Copy link
Collaborator

Choose a reason for hiding this comment

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

末尾换行不要删除

``` sh
python cfdgcn.py mode=export
```

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change

``` sh
python cfdgcn.py mode=infer
```

Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change

Comment on lines +176 to +178
input_spec = [
{key: InputSpec([None, 1], "float32", name=key) for key in model.input_keys},
]
Copy link
Collaborator

Choose a reason for hiding this comment

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

cfdgcn的输入是[None, 1]的形状吗?感觉这里不太对

Comment on lines +184 to +185

# 初始化预测器
Copy link
Collaborator

Choose a reason for hiding this comment

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

换行保留,但是中文注释可以删除

@luotao1 luotao1 added the HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务 label Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants