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

Skip to content

Modify data prep for mSUPERB multilingual#4965

Merged
mergify[bot] merged 9 commits intoespnet:masterfrom
guapaQAQ:mSUPERB
Feb 26, 2023
Merged

Modify data prep for mSUPERB multilingual#4965
mergify[bot] merged 9 commits intoespnet:masterfrom
guapaQAQ:mSUPERB

Conversation

@guapaQAQ
Copy link
Contributor

No description provided.

@mergify mergify bot added the ESPnet2 label Feb 25, 2023
train_set=data/train_${duration}${suffix}
cut -f 2- ${train_set}/text | grep -o -P '\[.*?\]|\<.*?\>' | sort | uniq > ${nlsyms_txt}
log "save non-linguistic symbols in ${nlsyms_txt}"
log "no non-linguistic symbols needed"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why do we disable this part?

@codecov
Copy link

codecov bot commented Feb 26, 2023

Codecov Report

Merging #4965 (09e3b54) into master (54f9610) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4965   +/-   ##
=======================================
  Coverage   77.10%   77.10%           
=======================================
  Files         604      604           
  Lines       53611    53611           
=======================================
  Hits        41338    41338           
  Misses      12273    12273           
Flag Coverage Δ
test_integration_espnet1 66.33% <ø> (ø)
test_integration_espnet2 47.50% <ø> (ø)
test_python 66.96% <ø> (ø)
test_utils 23.35% <ø> (ø)

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

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

@ftshijt ftshijt added the auto-merge Enable auto-merge label Feb 26, 2023
@@ -1,5 +1,6 @@
import argparse
import os
import string
Copy link
Collaborator

Choose a reason for hiding this comment

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

You forgot to import re

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks

@mergify mergify bot merged commit 5455c7c into espnet:master Feb 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Enable auto-merge ESPnet2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants