Skip to content

Replace monotonic-align from source by our published ilt-monotonic-align#812

Merged
joanise merged 2 commits into
mainfrom
dev.ej/monotonic-align-fork
May 27, 2026
Merged

Replace monotonic-align from source by our published ilt-monotonic-align#812
joanise merged 2 commits into
mainfrom
dev.ej/monotonic-align-fork

Conversation

@joanise
Copy link
Copy Markdown
Member

@joanise joanise commented May 27, 2026

PR Goal?

Get ilt-monotonic-align from PyPI instaed of github, so we can publish the next EveryVoice version.

This is a draft to review the publication process of ilt-monotonic-align, which is currently only published on test.pypi.org.
Please test with this PR and if we're happy with it, then I'll actually publish to regular PyPI.

Source code for our ilt-monotonic-align fork: https://github.com/EveryVoiceTTS/monotonic_align

Fixes?

Fixes EveryVoiceTTS/StyleTTS2#8

Feedback sought?

Please test and make sure it works for you.

Also please review
EveryVoiceTTS/monotonic_align@c6e5e6c...783173d
which is the diff between the repo we forked and what I have done, specifically for how I declare Copyright and maintainer, and general sanity checking. The intent at this point was to do the minimal changes required to be able to publish, so ignore "would-be-nice" elements, just review for actual problems.

Priority?

before we publish 0.5.0

Tests added?

yes

How to test?

By default, pip or uv won't fetch packages from testpypi, you need to add --index-strategy unsafe-best-match --extra-index-url https://test.pypi.org/simple to make that happen.

The lock file uv.lock file is already updated to point to testpypi for this one package, so just

uv sync --locked

will work.
(Once we publish ilt-monotonic-align to PyPI, I'll update the lock file again accordingly.)

Otherwise, you can do

uv pip install --index-strategy unsafe-best-match --extra-index-url https://test.pypi.org/simple -e .

Then, launch training of a styletts2 model and make sure it gets past the point where it need monotonic-align. @roedoejet when is that anyway?

Confidence?

high

Version change?

no

Related PRs?

EveryVoiceTTS/StyleTTS2#10

@joanise joanise requested a review from roedoejet May 27, 2026 17:08
@semanticdiff-com
Copy link
Copy Markdown

semanticdiff-com Bot commented May 27, 2026

Review changes with  SemanticDiff

Changed Files
File Status
  everyvoice/model/e2e/StyleTTS2_lightning  0% smaller
  pyproject.toml Unsupported file format
  uv.lock Unsupported file format

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 27, 2026

CLI load time: 0:00.22
Pull Request HEAD: 7f0d5d48a760d6bc9348014ec9aa87d899e66a37

Imports that take more than 0.1 s:
import time: self [us] | cumulative | imported package
import time:      6210 |     130426 |     typer.main
import time:       528 |     171556 |   typer
import time:     20262 |     114696 |         multiprocessing.context
import time:     19932 |     134627 |       multiprocessing
import time:       862 |     135489 |     everyvoice.base_cli.interfaces
import time:      1440 |     137134 |   everyvoice.base_cli.checkpoint
import time:    104061 |     160636 |                     concurrent.futures
import time:      6236 |     232556 |                   asyncio.base_events
import time:      6269 |     251741 |                 asyncio
import time:       296 |     252037 |               loguru._asyncio_loop
import time:      4668 |     330338 |             loguru._logger
import time:      2976 |     334327 |           loguru
import time:      2063 |     402903 |         everyvoice.utils
import time:       572 |     403474 |       everyvoice.model.e2e.StyleTTS2_lightning.styletts2.cli.preprocess
import time:       402 |     406080 |     everyvoice.model.e2e.StyleTTS2_lightning.styletts2.cli
import time:        15 |     406094 |   everyvoice.model.e2e.StyleTTS2_lightning.styletts2.cli.train
import time:      4624 |     828256 | everyvoice.cli
import time:    143752 |     178731 |                 email.message
import time:     58845 |     268219 |               importlib.metadata._adapters
import time:     36306 |     336192 |             importlib.metadata
import time:       352 |     336544 |           pygments.plugin
import time:      3382 |     343467 |         pygments.filters
import time:      3936 |     349077 |       pygments.lexer
import time:      4608 |     372612 |     rich.syntax
import time:      4406 |     448925 |   rich.markdown
import time:      3593 |     524802 | typer.rich_utils

@codecov
Copy link
Copy Markdown

codecov Bot commented May 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.55%. Comparing base (9b4e7cb) to head (7f0d5d4).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #812   +/-   ##
=======================================
  Coverage   87.55%   87.55%           
=======================================
  Files          45       45           
  Lines        4033     4033           
  Branches      605      605           
=======================================
  Hits         3531     3531           
  Misses        365      365           
  Partials      137      137           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Member

@roedoejet roedoejet left a comment

Choose a reason for hiding this comment

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

tested with a newly built env, can confirm it works. thanks so much @joanise !

@joanise joanise force-pushed the dev.ej/monotonic-align-fork branch from 098d763 to 3475ac6 Compare May 27, 2026 19:42
@joanise joanise force-pushed the dev.ej/monotonic-align-fork branch from 3475ac6 to 7f0d5d4 Compare May 27, 2026 19:59
@joanise joanise merged commit 7f0d5d4 into main May 27, 2026
10 checks passed
@joanise joanise deleted the dev.ej/monotonic-align-fork branch May 27, 2026 20:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fork monotonic-align and publish it to PyPI

2 participants