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

Update preprocessor.py #5200

Merged
merged 1 commit into from Jun 1, 2023
Merged

Update preprocessor.py #5200

merged 1 commit into from Jun 1, 2023

Conversation

sw005320
Copy link
Contributor

@sw005320 sw005320 commented Jun 1, 2023

relaxing the maximum output length warning (from 100 to 500 token length)

@sw005320 sw005320 added this to the v.202307 milestone Jun 1, 2023
@mergify mergify bot added the ESPnet2 label Jun 1, 2023
@codecov
Copy link

codecov bot commented Jun 1, 2023

Codecov Report

Merging #5200 (f03539f) into master (e6a628e) will increase coverage by 10.18%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master    #5200       +/-   ##
===========================================
+ Coverage   64.35%   74.54%   +10.18%     
===========================================
  Files         640      640               
  Lines       57057    57265      +208     
===========================================
+ Hits        36719    42688     +5969     
+ Misses      20338    14577     -5761     
Flag Coverage Δ
test_integration_espnet1 66.28% <ø> (?)
test_integration_espnet2 47.58% <0.00%> (?)
test_python 65.28% <100.00%> (+0.92%) ⬆️
test_utils 23.28% <ø> (?)

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

Impacted Files Coverage Δ
espnet2/train/preprocessor.py 29.16% <100.00%> (+13.15%) ⬆️

... and 198 files with indirect coverage changes

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

@sw005320 sw005320 merged commit 250f656 into master Jun 1, 2023
24 of 26 checks passed
@sw005320 sw005320 deleted the sw005320-patch-2 branch June 1, 2023 23:21
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.

None yet

1 participant