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

Add new method for custom formatter handling #293

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

ivuorio
Copy link

@ivuorio ivuorio commented Feb 7, 2025

Adding a method for setting new formatter into the native formatter handling. This does not change how the existing solutions work, and thus should not cause any backwards compatibility issue. But also allows user to implement multiple custom formatters and allows using them in datasets according to their name just like the builtin formatters. This solves both problems from the issue #290 as the the custom formatter is handled just like builtin ones.

Adding:

  • New method for adding custom formatters
  • Updating the documentation about the custom formatter using

Testing:

  • Manually tested with multiple datasets in YourTTS model training. Works with both compute embeddings and dataset loader.
  • Test pass.

ToDo:

  • Add unit tests for the new method
  • Add unit tests for custom formatter

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