Skip to content

Conversation

@yaqiangsun
Copy link
Contributor

@yaqiangsun yaqiangsun commented Apr 8, 2025

Fix index bug for doccano convert

Before submitting

  • Lint code. If there are lint issues, please format the code first.
# Install and register `pre-commit` in the project folder
pip install pre-commit && pre-commit install

# Process previous code files separately
pre-commit run --file XXXX.py
  • Add test cases into tests folder. If there are codecov issues, please add tests cases first.

PR types

Bug fixes

PR changes

Others

Description

Fix wrong index bug.

Fix index bug for doccano convert
@paddle-bot
Copy link

paddle-bot bot commented Apr 8, 2025

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Apr 8, 2025

CLA assistant check
All committers have signed the CLA.

@DrownFish19 DrownFish19 changed the title Fix wrong index bug in utils.py for docccano covert [Application] Fix wrong index bug in utils.py for docccano covert Apr 9, 2025
Copy link
Collaborator

@DrownFish19 DrownFish19 left a comment

Choose a reason for hiding this comment

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

LGTM

@DrownFish19
Copy link
Collaborator

Paddle修改导致PaddleNLP历史写法失效,现在已经修改。等待#10364 合入develop分支后拉取最新分支。

@codecov
Copy link

codecov bot commented Apr 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.01%. Comparing base (1d5591b) to head (f571226).
Report is 190 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #10361      +/-   ##
===========================================
- Coverage    49.02%   49.01%   -0.01%     
===========================================
  Files          765      765              
  Lines       125879   125879              
===========================================
- Hits         61709    61699      -10     
- Misses       64170    64180      +10     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yaqiangsun
Copy link
Contributor Author

@DrownFish19 @lugimzzz 请帮助Merge该PR

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Apr 22, 2025
Copy link
Collaborator

@DrownFish19 DrownFish19 left a comment

Choose a reason for hiding this comment

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

LGTM

@DrownFish19 DrownFish19 merged commit 8e812f6 into PaddlePaddle:develop Apr 22, 2025
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants