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

Added softplus + activation oneDNN fuse pass #36657

Merged

Conversation

jakpiase
Copy link
Contributor

@jakpiase jakpiase commented Oct 22, 2021

PR types

New features

PR changes

OPs

Describe

Added softplus + activation oneDNN fuse pass. It improved ppyolov3_r50vd_dcn_365 performance by 0.7%

@paddle-bot-old
Copy link

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

@jakpiase
Copy link
Contributor Author

@tsocha Please review that PR

tsocha
tsocha previously approved these changes Oct 25, 2021
tsocha
tsocha previously approved these changes Oct 25, 2021
@jakpiase
Copy link
Contributor Author

@lidanqing-intel Could you please ask somebody from Baidu to approve PR-CI-Static-Check?

@lidanqing-intel
Copy link
Contributor

@baoachun Could you please help request following people to approve this PR-CI-Static-Check? Or should we change anything to make CI to pass itself? Thanks !

2021-10-27 19:50:15 ****************
2021-10-27 19:50:15 Please find RD for approval first, and then find TPM for approval.
2021-10-27 19:50:15 0. You must have  one TPM approval for API documents change:
2021-10-27 19:50:15  jzhang533/ZhangJun, dingjiaweiww/DingJiaWei, Heeenrrry/LiKunLun, TCChenlong/ChenLong for general API docs
2021-10-27 19:50:15  PangHua/XiangHui for distributed related API docs
2021-10-27 19:50:15  twismon/WangYunKai, CheQiXiao/CheQiXiao for inference related API docs.
``

jczaja
jczaja previously approved these changes Oct 29, 2021
@lidanqing-intel lidanqing-intel changed the title Added softplus + activation oneDNN fuse pass [No-docker-pre-commit] Added softplus + activation oneDNN fuse pass Nov 2, 2021
@lidanqing-intel lidanqing-intel changed the title [No-docker-pre-commit] Added softplus + activation oneDNN fuse pass [Need-docker-pre-commit] Added softplus + activation oneDNN fuse pass Nov 2, 2021
@lidanqing-intel lidanqing-intel changed the title [Need-docker-pre-commit] Added softplus + activation oneDNN fuse pass [pre-committed-but-CI-fail] Added softplus + activation oneDNN fuse pass Nov 3, 2021
@lidanqing-intel lidanqing-intel changed the title [pre-committed-but-CI-fail] Added softplus + activation oneDNN fuse pass [Pre-committed-but-CI-fail] Added softplus + activation oneDNN fuse pass Nov 3, 2021
@jakpiase jakpiase requested a review from jczaja November 4, 2021 09:06
@lidanqing-intel lidanqing-intel changed the title [Pre-committed-but-CI-fail] Added softplus + activation oneDNN fuse pass [Waiting email] Added softplus + activation oneDNN fuse pass Nov 4, 2021
@jczaja
Copy link
Contributor

jczaja commented Nov 8, 2021

@baoachun, @lidanqing-intel Please help to have this PR approved so it pass CI Static Check

@baoachun
Copy link
Contributor

baoachun commented Nov 9, 2021

@baoachun, @lidanqing-intel Please help to have this PR approved so it pass CI Static Check

I have communicated with my colleague, but he has not yet approve this pr, I will urge it tomorrow.

@paddle-bot-old
Copy link

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

@jakpiase jakpiase changed the title [Waiting email] Added softplus + activation oneDNN fuse pass Added softplus + activation oneDNN fuse pass Nov 10, 2021
@baoachun
Copy link
Contributor

Hi @jczaja, could you please merge the latest develop branch and push again? My colleague said that the reason for the failure may be that the latest code was not pulled.

@lidanqing-intel
Copy link
Contributor

@baoachun Now only one required CI not passing: PR-CE-Framework, restarted and wait for results.

@lidanqing-intel lidanqing-intel merged commit a346c4d into PaddlePaddle:develop Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants