8000 Fix Whisper tokenizer CI error by slSeanWU · Pull Request #5004 · espnet/espnet · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix Whisper tokenizer CI error #5004

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 14, 2023
Merged

Fix Whisper tokenizer CI error #5004

merged 1 commit into from
Mar 14, 2023

Conversation

slSeanWU
Copy link
Contributor

Install openai-whisper==20230308 to use huggingface tokenizer

@mergify mergify bot added the Installation label Mar 14, 2023
@codecov
Copy link
codecov bot commented Mar 14, 2023 •
edited
Loading

Codecov Report

Merging #5004 (07d7c07) into master (418418c) will increase coverage by 10.44%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master    #5004       +/-   ##
===========================================
+ Coverage   66.55%   77.00%   +10.44%     
===========================================
  Files         606      606               
  Lines       53554    53748      +194     
===========================================
+ Hits        35643    41388     +5745     
+ Misses      17911    12360     -5551
Flag Coverage Δ
test_integration_espnet1 66.29% <ø> (?)
test_integration_espnet2 47.76% <ø> (?)
test_python 66.84% <ø> (+0.99%) ⬆️
test_utils 23.28% <ø> (+0.26%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 189 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sw005320 sw005320 added the CI Travis, Circle CI, etc label Mar 14, 2023
@sw005320 sw005320 added this to the v.202303 milestone Mar 14, 2023
@sw005320 sw005320 merged commit 611a291 into espnet:master Mar 14, 2023
@sw005320
Copy link
Contributor

Thanks for the quick fix!

@sw005320 sw005320 mentioned this pull request Mar 15, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CI Travis, Circle CI, etc Installation
Projects
None yet
Milestone
v.202303
Development

Successfully merging this pull request may close these issues.
2 participants
@slSeanWU @sw005320
0