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.

Sign up for GitHub

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

Jump to bottom

Switch to forked coqpit #10

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Switch to forked coqpit #10

wants to merge 5 commits into from
+70 −54

Conversation

eginhard
Copy link
Member

@eginhard eginhard commented Oct 19, 2024 •
edited
Loading

  • Switch to our forked coqpit-config package
  • Adjust some function parameters and type hints accordingly
  • Add some additional tests to ensure everything still works as before

coqui-tts is restricted to coqui-tts-trainer<0.2.0 so won't switch automatically yet in case of any issues.

@eginhard eginhard mentioned this pull request Oct 19, 2024
Merged
@eginhard eginhard force-pushed the coqpit branch 7 times, most recently from bd2afe9 to 900b22d Compare November 11, 2024 01:13
@eginhard eginhard force-pushed the coqpit branch 2 times, most recently from a284713 to b6ca19a Compare November 20, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@eginhard