Skip to content

Conversation

@SevenSamon
Copy link
Contributor

@SevenSamon SevenSamon commented Aug 13, 2024

PR types

Bug fixes|修改ChineseBertTokenizer不存在Max_length参数问题,以及pypinyin的import方式修改

PR changes

Models|paddlenlp/transformers/chinesebert/tokenizer.py

Description

修改ChineseBertTokenizer不存在Max_length参数问题,以及pypinyin的import方式修改

@paddle-bot
Copy link

paddle-bot bot commented Aug 13, 2024

Thanks for your contribution!

@DrownFish19 DrownFish19 changed the title ChineseBertTokenizer_max_length-problem [tokenizer] fix ChineseBertTokenizer with max_length/max_seq_len Aug 13, 2024
@SevenSamon SevenSamon closed this Aug 13, 2024
@SevenSamon SevenSamon reopened this Aug 13, 2024
@codecov
Copy link

codecov bot commented Aug 13, 2024

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.00%. Comparing base (ecfe0ac) to head (6915194).
⚠️ Report is 960 commits behind head on develop.

⚠️ Current head 6915194 differs from pull request most recent head 4c265bb

Please upload reports for the commit 4c265bb to get more accurate results.

Files with missing lines Patch % Lines
paddlenlp/transformers/chinesebert/tokenizer.py 0.00% 4 Missing ⚠️

❌ Your patch status has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.
❌ Your project status has failed because the head coverage (55.00%) is below the target coverage (58.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8926      +/-   ##
===========================================
- Coverage    55.01%   55.00%   -0.02%     
===========================================
  Files          646      646              
  Lines       102242   102030     -212     
===========================================
- Hits         56252    56124     -128     
+ Misses       45990    45906      -84     

☔ 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.

@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Aug 13, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Aug 13, 2024
DrownFish19
DrownFish19 previously approved these changes Aug 19, 2024
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Aug 22, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Aug 22, 2024
@github-actions
Copy link

This Pull Request is stale because it has been open for 60 days with no activity. 当前Pull Request 60天内无活动,被标记为stale。

@github-actions github-actions bot added stale and removed stale labels Oct 23, 2024
@github-actions
Copy link

This Pull Request is stale because it has been open for 60 days with no activity. 当前Pull Request 60天内无活动,被标记为stale。

@github-actions github-actions bot added the stale label Dec 23, 2024
@paddle-bot paddle-bot bot closed this Dec 30, 2025
@paddle-bot
Copy link

paddle-bot bot commented Dec 30, 2025

Automatically closed by Paddle-bot.

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.

2 participants