Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Performance] Compatible with flashmask API rename upgrade #9019

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

GuoxiaWang
Copy link
Contributor

PR types

Function optimization

PR changes

APIs

Description

compatible with flashmask API rename upgrade

Copy link

paddle-bot bot commented Aug 27, 2024

Thanks for your contribution!

Copy link
Collaborator

@wawltor wawltor left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

codecov bot commented Aug 27, 2024

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 54.12%. Comparing base (f6fc7ff) to head (4b5d739).
Report is 345 commits behind head on develop.

Files with missing lines Patch % Lines
paddlenlp/transformers/llama/fusion_ops.py 0.00% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9019      +/-   ##
===========================================
+ Coverage    54.05%   54.12%   +0.06%     
===========================================
  Files          650      650              
  Lines       103884   103886       +2     
===========================================
+ Hits         56155    56227      +72     
+ Misses       47729    47659      -70     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@ZHUI ZHUI changed the title compatible with flashmask API rename upgrade [Performance] Compatible with flashmask API rename upgrade Sep 13, 2024
@ZeyuChen ZeyuChen merged commit 1fc9429 into PaddlePaddle:develop Sep 30, 2024
9 of 12 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants