Several improvements to wav2vec2aligner system: early errors, testing, py 3.8 compatibility, etc

[joanise]

PR Goal?

This started just fixing EveryVoiceTTS/EveryVoice#326 but then I saw there was no unit testing, so it went on to include several related things needed to test my fix for this issue:

• unit testing for this module's code
• enabling that unit testing in CI
• fix the python 3.8 incompatibilities I noticed in the process
• add missing dependencies
• remove the broken main guard block and replace it by a valid __main__.py
• fix deprecation warnings (spliced into refactor: change deprecated autocompletion to shell_complete #15)

Fixes?

Fixes EveryVoiceTTS/EveryVoice#326

Feedback sought?

Regular code review.

Please look at the individual commits if you want the reason for each separate change.

Priority?

beta

Tests added?

Yes!

How to test?

For 326:

• ctc-segmenter align /dev/null /dev/null (or everyvoice segment align /dev/null /dev/null) will now give a friendly error message instead of an exception and stack dump.

For the tests:

• python -m unittest discover aligner.tests will run the new unit tests (you need to do pip install -e . in this module first).

Confidence?

high

Version change?

no

Related PRs?

EveryVoiceTTS/EveryVoice#590

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
aligner/heavy.py	37% smaller
aligner/cli.py	15% smaller
aligner/utils.py	6% smaller
.coveragerc	Unsupported file format
.github/workflows/test.yml	Unsupported file format
.gitignore	Unsupported file format
aligner/__main__.py	0% smaller
aligner/tests/__init__.py	0% smaller
aligner/tests/test_cli.py	0% smaller
requirements.txt	Unsupported file format

[github-actions]

CLI load time: 0:00.16
Pull Request HEAD: 0f41adcf3180a7f7b5f5ebf076e22d1e5ab697df
Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package
import time:      1145 |     107454 |     typer.main
import time:       303 |     128664 |   typer
import time:       405 |     132636 | aligner.cli

[codecov]

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

[SamuelLarkin]

Approved but I have a question about 7.