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

Skip to content

Fix a 'ref_channel'-related issue in espnet2/bin/enh_inference.py#4972

Merged
sw005320 merged 1 commit intoespnet:masterfrom
Emrys365:se_utils
Feb 28, 2023
Merged

Fix a 'ref_channel'-related issue in espnet2/bin/enh_inference.py#4972
sw005320 merged 1 commit intoespnet:masterfrom
Emrys365:se_utils

Conversation

@Emrys365
Copy link
Collaborator

Fix the issue reported in #4970.

@Emrys365 Emrys365 added Bugfix ESPnet2 SE Speech enhancement labels Feb 28, 2023
@codecov
Copy link

codecov bot commented Feb 28, 2023

Codecov Report

Merging #4972 (4e3e14e) into master (84ccae7) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           master    #4972   +/-   ##
=======================================
  Coverage   77.09%   77.10%           
=======================================
  Files         604      604           
  Lines       53611    53613    +2     
=======================================
+ Hits        41332    41338    +6     
+ Misses      12279    12275    -4     
Flag Coverage Δ
test_integration_espnet1 66.33% <ø> (ø)
test_integration_espnet2 47.50% <0.00%> (+0.16%) ⬆️
test_python 66.95% <0.00%> (-0.01%) ⬇️
test_utils 23.35% <ø> (ø)

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

Impacted Files Coverage Δ
espnet2/bin/enh_inference.py 91.16% <0.00%> (-0.74%) ⬇️
espnet2/train/trainer.py 77.33% <0.00%> (+0.93%) ⬆️
espnet2/train/gan_trainer.py 69.58% <0.00%> (+1.03%) ⬆️

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

@sw005320 sw005320 added this to the v.202303 milestone Feb 28, 2023
@sw005320
Copy link
Contributor

Thanks for the quick fix!

@sw005320 sw005320 merged commit 38e1320 into espnet:master Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bugfix ESPnet2 SE Speech enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants