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

Fix offset mapping for uie-x #4687

Merged
merged 1 commit into from
Feb 13, 2023
Merged

Conversation

linjieccc
Copy link
Contributor

PR types

Bug fixes

PR changes

APIs

Description

@paddle-bot
Copy link

paddle-bot bot commented Feb 7, 2023

Thanks for your contribution!

@codecov
Copy link

codecov bot commented Feb 7, 2023

Codecov Report

Merging #4687 (0b03a0f) into develop (67d82fd) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##           develop    #4687   +/-   ##
========================================
  Coverage    42.08%   42.09%           
========================================
  Files          434      434           
  Lines        61870    61915   +45     
========================================
+ Hits         26039    26063   +24     
- Misses       35831    35852   +21     
Impacted Files Coverage Δ
paddlenlp/taskflow/information_extraction.py 8.08% <0.00%> (-0.03%) ⬇️
paddlenlp/transformers/image_processing_utils.py 24.41% <0.00%> (-1.59%) ⬇️
paddlenlp/transformers/mbart/tokenizer.py 89.69% <0.00%> (-0.78%) ⬇️
paddlenlp/transformers/t5/tokenizer.py 91.26% <0.00%> (-0.66%) ⬇️
paddlenlp/transformers/clip/tokenizer.py 95.30% <0.00%> (-0.57%) ⬇️
paddlenlp/transformers/model_utils.py 77.40% <0.00%> (-0.35%) ⬇️
paddlenlp/transformers/tokenizer_utils_base.py 78.14% <0.00%> (-0.24%) ⬇️
paddlenlp/transformers/tokenizer_utils.py 63.08% <0.00%> (+0.14%) ⬆️
paddlenlp/utils/downloader.py 62.99% <0.00%> (+0.44%) ⬆️
paddlenlp/transformers/ernie_m/tokenizer.py 82.14% <0.00%> (+0.71%) ⬆️
... and 4 more

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

Copy link
Collaborator

@sijunhe sijunhe left a comment

Choose a reason for hiding this comment

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

information_extraction taskflow这里逻辑很重,希望能加上单测

@linjieccc
Copy link
Contributor Author

information_extraction taskflow这里逻辑很重,希望能加上单测

好的,后续会针对uie相关功能增加下对应单测

@linjieccc linjieccc merged commit c453f51 into PaddlePaddle:develop Feb 13, 2023
@linjieccc linjieccc deleted the fix-uie-x branch February 13, 2023 08:19
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.

None yet

2 participants