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 variants in favour of #1649 #1677

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hozosch
Copy link
Contributor

@hozosch hozosch commented Mar 12, 2023

I'm sorry, I didn't know it would detect all variants as changed, I hope it's ok like this. If this pull request is doing any harm, please let me know. I don't see any harm in adding these variants as long as they're handed in the correct way.

I'm sorry, I didn't know it would detect all variants as changed, I hope it's ok like this. If this pull request is doing any harm, please let me know. I don't see any harm in adding these variants as long as they're handed in the correct way.
@hozosch
Copy link
Contributor Author

hozosch commented Mar 12, 2023

Again, I'm sorry, I don't know how to fix the checks. There is one successful one and the rest failed, but I don't know what to do here.

@hozosch hozosch mentioned this pull request Mar 12, 2023
@jaacoppi
Copy link
Collaborator

Increasing N_VOICES_LIST passes make check. Probably passes other CI tests as well.

The current value seems arbitrary so it could be changed easily. If we simply increase the value, this same problem will occur later. Should we try to find a better solution by dynamically setting the array size?

@hozosch
Copy link
Contributor Author

hozosch commented Mar 12, 2023

Hmm, don't know about that, but I think I kind of understand. So some file must be changed? Don't know how you would make that array dynamic, but if we don't want that problem coming up any longer, this sounds to me as the right solution.

@gregodejesus
Copy link

gregodejesus commented Jun 2, 2023

hello. I think this should solve the problems of eSpeak ng when more than 200 variants are added, remove some duplicates so that they do not enter into conflicts with the languages. Here I will share a package with the necessary variants to be added. Its 166 variants.
without further ado, make the package from here
!v.zip

@gregodejesus
Copy link

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.

None yet

3 participants