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

Added RU support #2

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Added RU support #2

wants to merge 4 commits into from

Conversation

kosnag
Copy link

@kosnag kosnag commented Oct 3, 2024

No description provided.

@SethClydesdale SethClydesdale added the enhancement New feature or request label Oct 3, 2024
@pp0236
Copy link

pp0236 commented Oct 3, 2024

🤔

@SethClydesdale
Copy link
Owner

Hello, @kosnag!

I think expanding language support is a good idea, so I really appreciate this! The only issue I notice right now is that there are conflicts that need to be addressed. That is, encoding as English and then decoding produces Russian. I can think of a few ways to resolve this, such as:

  • Creating a Russian instance of this tool under a /ru/ sub-directory
  • Adding a language selector for encoding/decoding that switches between the language in the Arona.cunny object (e.g. Arona.cunny['en'].encoder, Arona.cunny['ru'].encoder, etc.)

I think the former might be the best solution as it'd allow you to also translate the interface into your language. You could also most likely run off the main arona.js with an additional language file that overwrites content in Arona.cunny and Arona.speech to help simplify things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants