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

【PFCC算子性能优化】添加selu算子性能优化文档 #169

Merged
merged 4 commits into from
Jul 19, 2022

Conversation

carryyu
Copy link
Contributor

@carryyu carryyu commented Jul 6, 2022

No description provided.

@CLAassistant
Copy link

CLAassistant commented Jul 6, 2022

CLA assistant check
All committers have signed the CLA.

@carryyu
Copy link
Contributor Author

carryyu commented Jul 6, 2022 via email

| 1 | [8, 1024, 3073] |0.4630 |
| 2 | [5100, 38506] | 3.6001|
| 3 | [300, 100, 128] |0.073 |

Copy link
Contributor

Choose a reason for hiding this comment

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

这部分感觉可以罗列一下 paddle目前的性能状态,以及对应的case配置信息

@carryyu carryyu closed this Jul 13, 2022
@carryyu carryyu reopened this Jul 13, 2022
@carryyu carryyu closed this Jul 13, 2022
@Ligoml
Copy link
Contributor

Ligoml commented Jul 14, 2022

为什么close了呀?

@carryyu
Copy link
Contributor Author

carryyu commented Jul 14, 2022

为什么close了呀?

那个性能测试有一些出入,考虑重新测试后再PR。

@luotao1
Copy link
Collaborator

luotao1 commented Jul 14, 2022

那个性能测试有一些出入,考虑重新测试后再PR

可以不关闭PR,再提一个commit即可

@carryyu carryyu changed the title 添加selu算子性能优化文档 【PFCC算子性能优化】添加selu算子性能优化文档 Jul 14, 2022
@carryyu carryyu reopened this Jul 14, 2022
@carryyu
Copy link
Contributor Author

carryyu commented Jul 14, 2022

我重新commit了,请问必须该PR通过后才能提交算子代码到Paddle仓库吗。

|---|---|---|
| 1 | 理清Paddle中OP设计思路,同类产品中最佳设计方案 | 2022-07-06 |
| 2 | 完成开发文档设计 | 2022-07-07 |
| 3 | 完成代码开发工作,并通过线程CI测试 | 2022-07-10 |
Copy link
Contributor

Choose a reason for hiding this comment

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

文档整体我觉得OK了,目前就是这个时间规划应该需要结合当前的进展改一下哈。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

已修改。

JamesLim-sy
JamesLim-sy previously approved these changes Jul 14, 2022
Copy link
Contributor

@JamesLim-sy JamesLim-sy left a comment

Choose a reason for hiding this comment

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

LGTM, good work!

@carryyu
Copy link
Contributor Author

carryyu commented Jul 14, 2022

LGTM, good work!

修复了case序号上的一个小问题,抱歉!

@luotao1
Copy link
Collaborator

luotao1 commented Jul 15, 2022

我重新commit了,请问必须该PR通过后才能提交算子代码到Paddle仓库吗

这个PR没通过也可以提交算子代码到Paddle仓库,但review会先等RFC通过后再review Paddle仓库的PR

@carryyu carryyu requested a review from JamesLim-sy July 17, 2022 06:31
Copy link
Contributor

@JamesLim-sy JamesLim-sy left a comment

Choose a reason for hiding this comment

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

LGTM, good work.

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.

5 participants