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

Skip to content

CI test for s2st#5526

Merged
mergify[bot] merged 16 commits intoespnet:masterfrom
ftshijt:ci_s2st
Nov 9, 2023
Merged

CI test for s2st#5526
mergify[bot] merged 16 commits intoespnet:masterfrom
ftshijt:ci_s2st

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
@codecov
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
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.

2 participants