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

Show original model URLs #149

Merged
merged 4 commits into from
Nov 11, 2024
Merged

Show original model URLs #149

merged 4 commits into from
Nov 11, 2024

Conversation

eginhard
Copy link
Member

@eginhard eginhard commented Nov 10, 2024

The scarf.sh-redirect URLs were only needed for Coqui to collect statistics. It's clearer to show the original URLs, which can be obtained with e.g. wget --spider --max-redirect=1 https://coqui.gateway.scarf.sh/hf/bark/coarse_2.pt

In CI:

  • deduplicate by moving setup-uv into a composite action
  • run the slower integration tests only for the lowest and highest supported Python version to reduce the number of CI checks and total running time

@eginhard eginhard force-pushed the cache-models branch 5 times, most recently from 9ac105e to aa42b27 Compare November 10, 2024 21:11
These allowed Coqui to get download stats, which we don't need anymore
@eginhard eginhard changed the title Cache TTS models Show original model URLs Nov 10, 2024
More breaking changes affecting the XTTS streaming code
@eginhard eginhard merged commit b5bd995 into dev Nov 11, 2024
36 checks passed
@eginhard eginhard deleted the cache-models branch November 11, 2024 12:24
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.

2 participants