Skip to content

Dev.ap/stts2 hf checkpoints#806

Closed
roedoejet wants to merge 4 commits into
mainfrom
dev.ap/stts2-hf-checkpoints
Closed

Dev.ap/stts2 hf checkpoints#806
roedoejet wants to merge 4 commits into
mainfrom
dev.ap/stts2-hf-checkpoints

Conversation

@roedoejet
Copy link
Copy Markdown
Member

PR Goal?

Loading pretrained models from within the codebase is messy since we need to handle the paths, and the model checkpoints bloat the package/our git history.

Fixes?

An element of #686

Feedback sought?

Does this look reasonable? I think the "local override" option is still not going to be handled well since it will put a potentially absolute path in the config.

Priority?

high

Tests added?

None

How to test?

if you train a model with this branch, you should see the HF models downloading automatically

Confidence?

medium

Version change?

n/a

Related PRs?

EveryVoiceTTS/StyleTTS2#6

Once accepted, we'll need to clean the git history of any *.t7 or *.pth files

@semanticdiff-com
Copy link
Copy Markdown

semanticdiff-com Bot commented May 19, 2026

Review changes with  SemanticDiff

Changed Files
File Status
  everyvoice/tests/test_cli.py  68% smaller
  everyvoice/cli.py  2% smaller
  everyvoice/base_cli/checkpoint.py  0% smaller
  everyvoice/base_cli/prediction_writing_callback.py  0% smaller
  everyvoice/demo/app.py  0% smaller
  everyvoice/model/e2e/StyleTTS2_lightning  0% smaller
  everyvoice/model/feature_prediction/FastSpeech2_lightning  0% smaller

@roedoejet roedoejet requested a review from joanise May 19, 2026 23:33
@roedoejet roedoejet closed this May 19, 2026
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.

1 participant