Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Quick fix for whisper specaug#5206

Merged
ftshijt merged 1 commit intoespnet:masterfrom
siddhu001:whisper_specaug_issue
Jun 2, 2023
Merged

Quick fix for whisper specaug#5206
ftshijt merged 1 commit intoespnet:masterfrom
siddhu001:whisper_specaug_issue

Conversation

@siddhu001
Copy link
Collaborator

The specaug applied in whisper encoder has the issue mentioned here - #5198. This PR fixes it

@mergify mergify bot added the ESPnet2 label Jun 2, 2023
@sw005320 sw005320 added Bugfix auto-merge Enable auto-merge labels Jun 2, 2023
@sw005320 sw005320 added this to the v.202307 milestone Jun 2, 2023
@sw005320
Copy link
Contributor

sw005320 commented Jun 2, 2023

Thanks for your quick action!

@codecov
Copy link

codecov bot commented Jun 2, 2023

Codecov Report

Merging #5206 (13cbfc7) into master (279ccfe) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #5206      +/-   ##
==========================================
- Coverage   74.54%   74.54%   -0.01%     
==========================================
  Files         640      640              
  Lines       57265    57261       -4     
==========================================
- Hits        42688    42683       -5     
- Misses      14577    14578       +1     
Flag Coverage Δ
test_integration_espnet1 66.28% <ø> (ø)
test_integration_espnet2 47.58% <0.00%> (+<0.01%) ⬆️
test_python 65.27% <0.00%> (-0.01%) ⬇️
test_utils 23.28% <ø> (ø)

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

Impacted Files Coverage Δ
espnet2/asr/encoder/whisper_encoder.py 80.45% <0.00%> (-1.90%) ⬇️

... and 5 files with indirect coverage changes

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

@ftshijt ftshijt merged commit af3c582 into espnet:master Jun 2, 2023
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.

3 participants