-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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 Uzbek translation #478
Closed
Closed
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
a9486e4
Added Uzbek language
ismoilovdevml 9bca70f
in progress uz.po
ismoilovdevml dbdaab8
updated
ismoilovdevml 8482dd3
Merge branch 'google:main' into main
ismoilovdevml e4f0cff
in progress
ismoilovdevml cbbf4fd
updated
ismoilovdevml 047299c
Merge branch 'google:main' into main
ismoilovdevml 6bb0fc9
in progress
ismoilovdevml ce8c106
translation errors fixed
ismoilovdevml 99f6fb0
Merge branch 'google:main' into main
ismoilovdevml 2e5db4c
Merge branch 'google:main' into main
ismoilovdevml 968deb8
Merge branch 'google:main' into main
ismoilovdevml a138187
Merge branch 'google:main' into main
ismoilovdevml File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be "Tur Inferencei" or something similar that makes "Inference" or its translated substitute the head, right? Because it does not speak about a "Type of Inference" but vaguely "Inference of Types". I believe there are a few flips like this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sadly, Uzbek language doesn't have any translations for complex & scientific words and therefore we decided to translate only suffixes of words and adopt the base. In this example below, it would be way more correctly to translate it as "Inferensiya Turi". I do apologize for untranslated parts, @ismoilovdevml has been bugging out with this and currently we are working on single scientific words dictionary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katsuki-yuri The "-i" suffix in "Inferensiya Turi" is possessive person marker, right? Like "Inferensiyaning Turi". If that is the case, it does not produce the correct noun phrase, as in, English would be "The type of inference", which makes inference sound like a name of an object rather than a process.
"Tur Inferensiyasi" on the other hand produces the correct noun phrase.
Thank you very much for your kind reply
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just to exemplify, let's take a more widespread phrase: Conference attendance.
Konferentsiya ishtiroki is a corresponding translation.
The common aspect of "Conference attendance" and "Type inference" in English is head directionality parameter. Both phrases' head is the final word. Because Uzbek is a head-final language, the correct translation should emit corresponding head to the final position too. Hence, "Konferentsiya ishtiroki" and "Tur inferensiyasi" should be correct. "Ishtirok konferentsiyasi" and "Inferensiya turi" flip the words.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(just reminder that original phrase is "Type Inference" meaning "Inference of Types" or "Types' Inference" roughly)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm just writing my closing remarks in this review thread. If anybody familiar with translation process can send their dependency or constituency parse of "Tur inferensiyasi" and "Inferensiya turi" (in the assumed alphabet of this translation), it would be helpful for other eyes, but otherwise, I just wish you people good luck with improving digital resources for Uzbek people! Keep at it, open-source helps people ❤️ (and if possible please avoid ad-hominems and curse words to improve constructive quality of the dialogue)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
even if latin is being taught at schools, gov still uses cyrillic and follows hella another separate rules which can't be used in public. @mehmetoguzderin now you see the problem?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@katsuki-yuri I do recognize the struggle. But as I said in my last comment, I just have a final suggestion for the dialogue, otherwise, everything you all express here are well-noted and understood. Thank you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your understanding 🫱🏻🫲🏻
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! I would like to add something regarding the translation of "Type inference". After reading through all the thread, the confusion is in the understanding of the term of "Type Inference" itself in English. You can see the real difference between the terms here in these two examples: "the model is being used to do inference on types [Tur Inferensiyasi]" vs "there are around 10 inference types introduced in the last 2 years [Inferensiya Turlari]"
@mehmetoguzderin makes a good point in clarifying exactly which meaning of the phrase "Type Inference" being used here and I don't understand how the conversation about Latin and Cyrillic is relevant here at all.
But anyways, this is a small issue in itself and wouldn't probably make a big difference in the end. Thanks to @katsuki-yuri and @ismoilovdevml for the initiative!