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

Skip to content

Fix random segments#5274

Merged
ftshijt merged 3 commits intoespnet:masterfrom
iamanigeeit:fix_random_segments
Jul 21, 2023
Merged

Fix random segments#5274
ftshijt merged 3 commits intoespnet:masterfrom
iamanigeeit:fix_random_segments

Conversation

@iamanigeeit
Copy link
Contributor

@mergify mergify bot added the ESPnet2 label Jul 3, 2023
@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Merging #5274 (90b2522) into master (f122c22) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5274   +/-   ##
=======================================
  Coverage   76.10%   76.10%           
=======================================
  Files         658      658           
  Lines       59156    59157    +1     
=======================================
+ Hits        45022    45023    +1     
  Misses      14134    14134           
Flag Coverage Δ
test_integration_espnet1 65.96% <ø> (ø)
test_integration_espnet2 47.52% <100.00%> (+<0.01%) ⬆️
test_python 66.49% <100.00%> (+<0.01%) ⬆️
test_utils 23.17% <ø> (ø)

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

Impacted Files Coverage Δ
espnet2/gan_tts/utils/get_random_segments.py 100.00% <100.00%> (ø)

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

Copy link
Member

@kan-bayashi kan-bayashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is good idea. LGTM!

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.

5 participants