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

Use language with variants with biggest count of simulations. other should be removed. #211

Merged
merged 1 commit into from
Mar 19, 2023

Conversation

pavel-karatsiuba
Copy link
Collaborator

Changed logic to use --withoutLanguageVariants parameter.
When using --withoutLanguageVariants do not just remove languages with variants but use language with variants with the biggest count of simulations.
For example, if ku language has 41 simulations and ku_TR has 63 then for ku language use simulations from ku_TR. So in output, you will not see ku_TR but you will see ku with 63 simulations.

Fix: #207

@kelson42
Copy link
Contributor

@pavel-karatsiuba please rebase and fix conflict

@kelson42
Copy link
Contributor

@pavel-karatsiuba I'm very sorry, because of my being slow, I have to ask you a last time here to rebase and fix conflict.

@kelson42 kelson42 changed the title use language with variants with biggest count of simulations. other should be removed. Use language with variants with biggest count of simulations. other should be removed. Mar 12, 2023
@kelson42 kelson42 merged commit 78adbf0 into main Mar 19, 2023
@kelson42 kelson42 deleted the without-language-variants branch March 19, 2023 16:29
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.

More intelligent removing of language variants
2 participants