-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
ESPnet2 swbd_sentiment recipe #4134
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Many thanks! A few minor comments as attached
Codecov Report
@@ Coverage Diff @@
## master #4134 +/- ##
=======================================
Coverage 80.43% 80.43%
=======================================
Files 442 442
Lines 38557 38557
=======================================
Hits 31015 31015
Misses 7542 7542
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
LGTM! Will merge it after CI passed. Thanks! |
@ftshijt thanks for the review! |
This PR adds ESPnet2 swdb_sentiment recipe.