Skip to content

Conversation

@zjjlivein
Copy link
Collaborator

PR types

PR changes

Description

新增的fused_multi_transformer_encoder_pass 导致推理出现core dumped,该pass当前不支持前后权重共享的情况 PaddlePaddle/Paddle#49383

@paddle-bot
Copy link

paddle-bot bot commented Mar 6, 2023

Thanks for your contribution!

@codecov
Copy link

codecov bot commented Mar 6, 2023

Codecov Report

Merging #5117 (cbd344e) into develop (ce18df0) will not change coverage.
The diff coverage is n/a.

❗ Current head cbd344e differs from pull request most recent head c67d4ad. Consider uploading reports for the commit c67d4ad to get more accurate results

@@           Coverage Diff            @@
##           develop    #5117   +/-   ##
========================================
  Coverage    49.90%   49.90%           
========================================
  Files          456      456           
  Lines        65415    65415           
========================================
  Hits         32645    32645           
  Misses       32770    32770           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@zjjlivein zjjlivein requested a review from joey12300 March 6, 2023 10:37
Copy link
Contributor

@joey12300 joey12300 left a comment

Choose a reason for hiding this comment

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

LGTM

@joey12300 joey12300 merged commit ec29e4f into PaddlePaddle:develop Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants