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

Recipe fix for PueblaNahuatl Recipe #4522

Merged
merged 3 commits into from
Jul 21, 2022
Merged

Recipe fix for PueblaNahuatl Recipe #4522

merged 3 commits into from
Jul 21, 2022

Conversation

ftshijt
Copy link
Collaborator

@ftshijt ftshijt commented Jul 21, 2022

According to some direct contact, we found some minor issues in the Puebla Nahuatl recipe. Will be fixed with this PR.

@mergify mergify bot added the ESPnet2 label Jul 21, 2022
@codecov
Copy link

codecov bot commented Jul 21, 2022

Codecov Report

Merging #4522 (a982a04) into master (f2778f7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4522   +/-   ##
=======================================
  Coverage   82.40%   82.40%           
=======================================
  Files         481      481           
  Lines       41238    41238           
=======================================
  Hits        33982    33982           
  Misses       7256     7256           
Flag Coverage Δ
test_integration_espnet1 66.38% <ø> (ø)
test_integration_espnet2 49.17% <ø> (ø)
test_python 69.40% <ø> (ø)
test_utils 23.30% <ø> (ø)

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

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@ftshijt ftshijt merged commit 5599bc6 into master Jul 21, 2022
@ftshijt ftshijt deleted the ftshijt-patch-1 branch July 21, 2022 21:03
@kan-bayashi kan-bayashi added this to the v.202207 milestone Aug 2, 2022
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.

None yet

2 participants