Skip to content
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

CI test for s2st #5526

Merged
merged 16 commits into from
Nov 9, 2023
Merged

CI test for s2st #5526

merged 16 commits into from
Nov 9, 2023

Conversation

ftshijt
Copy link
Collaborator

@ftshijt ftshijt commented Nov 2, 2023

What?

CI test functions for S2ST

TODO

  • inference ci
  • mini_an4 integration

@ftshijt ftshijt self-assigned this Nov 2, 2023
@ftshijt ftshijt added Recipe CI Travis, Circle CI, etc S2ST Speech-to-speech translation ESPnet2 labels Nov 2, 2023
@ftshijt ftshijt added this to the v.202312 milestone Nov 2, 2023
Copy link

codecov bot commented Nov 2, 2023

Codecov Report

Merging #5526 (6137289) into master (3ba334e) will increase coverage by 1.62%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5526      +/-   ##
==========================================
+ Coverage   75.12%   76.75%   +1.62%     
==========================================
  Files         719      719              
  Lines       66425    66429       +4     
==========================================
+ Hits        49902    50986    +1084     
+ Misses      16523    15443    -1080     
Flag Coverage Δ
test_configuration_espnet2 ∅ <ø> (∅)
test_integration_espnet1 63.00% <ø> (ø)
test_integration_espnet2 50.14% <66.66%> (+1.05%) ⬆️
test_python_espnet1 19.10% <0.00%> (-0.01%) ⬇️
test_python_espnet2 52.60% <100.00%> (+1.38%) ⬆️
test_utils 22.19% <ø> (ø)

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

Files Coverage Δ
espnet2/bin/s2st_inference.py 64.19% <100.00%> (+64.19%) ⬆️
espnet2/tasks/s2st.py 86.31% <ø> (+86.31%) ⬆️

... and 24 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@ftshijt
Copy link
Collaborator Author

ftshijt commented Nov 6, 2023

Currently blocked by CI issues in multi-channel frontend, will resume after it is fixed

Copy link
Contributor

@sw005320 sw005320 left a comment

Choose a reason for hiding this comment

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

LGTM!
Thanks, @ftshijt!

@sw005320 sw005320 added the auto-merge Enable auto-merge label Nov 9, 2023
@mergify mergify bot merged commit 78e5a31 into espnet:master Nov 9, 2023
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge Enable auto-merge CI Travis, Circle CI, etc ESPnet2 Recipe S2ST Speech-to-speech translation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants