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

Fixed SQLalchemy version for MFA #5059

Merged
merged 2 commits into from Mar 25, 2023
Merged

Fixed SQLalchemy version for MFA #5059

merged 2 commits into from Mar 25, 2023

Conversation

Fhrozen
Copy link
Member

@Fhrozen Fhrozen commented Mar 25, 2023

Fix #5058

There is a new release of sqlalchemy that does not match MFA installers.
I will try to update later MFA version and setup its dependencies.

@mergify mergify bot added the Installation label Mar 25, 2023
@codecov
Copy link

codecov bot commented Mar 25, 2023

Codecov Report

Merging #5059 (ae61207) into master (c4d7063) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #5059   +/-   ##
=======================================
  Coverage   77.03%   77.03%           
=======================================
  Files         606      606           
  Lines       53755    53755           
=======================================
  Hits        41409    41409           
  Misses      12346    12346           
Flag Coverage Δ
test_integration_espnet1 66.29% <ø> (ø)
test_integration_espnet2 47.96% <ø> (ø)
test_python 66.86% <ø> (ø)
test_utils 23.28% <ø> (ø)

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

@sw005320 sw005320 added the auto-merge Enable auto-merge label Mar 25, 2023
@sw005320 sw005320 added this to the v.202303 milestone Mar 25, 2023
@sw005320
Copy link
Contributor

Thanks a lot!

BTW, can you also try alqalign?
We recently released a multilingual force aligner.
This shows good performance and also supports many (7000!) languages.

@mergify mergify bot merged commit af5512e into espnet:master Mar 25, 2023
24 checks passed
@Fhrozen Fhrozen deleted the pr-mfa branch April 6, 2023 13:01
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.

Unable to run mfa.sh to train fastspeech2
2 participants