-
Notifications
You must be signed in to change notification settings - Fork 353
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
CLDR-16970 Add testData file and generator for likely subtags #3173
CLDR-16970 Add testData file and generator for likely subtags #3173
Conversation
…de locales > Basic to cut down the size
# RemoveFavorScript: Remove Likely Subtags, when the script is favored. | ||
# Only included when different than AddLikely. | ||
# RemoveFavorRegion: Remove Likely Subtags, when the region is favored. | ||
# Only included when different than RemoveFavorScript. |
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.
Observation: I'm not sure we implement RemoveFavorScript
in ICU4X. Good to have this test data to find a deficiency in our feature coverage that we can work toward fixing. :)
It looks like it only makes a difference in a few cases (this is restricted to Basic locales):
|
I believe that's correct; these are the only instances where there is a
lang-region and a lang-script on the left side of the likely subtags mapping
<https://github.com/unicode-org/cldr-json/blob/main/cldr-json/cldr-core/supplemental/likelySubtags.json>
:
"sd-Deva": "sd-Deva-IN",
"sd-IN": "sd-Deva-IN",
and
"yue-CN": "yue-Hans-CN",
"yue-Hans": "yue-Hans-CN",
and
"zh-Hant": "zh-Hant-TW",
"zh-TW": "zh-Hant-TW",
I don't see a default region for "sr-Latn", for example, which means
"sr-Latn" gets expanded to "sr-Latn-RS", which cannot be minimized to
"sr-RS" because that would be changing the script.
…On Thu, Aug 10, 2023 at 5:29 PM Mark Davis ***@***.***> wrote:
It looks like it only makes a difference in a few cases (this is
restricted to Basic locales):
<style type="text/css"></style>
Source ; | AddLikely ; | RemoveFavorScript ; | RemoveFavorRegion
-- | -- | -- | --
sd-Deva ; | sd-Deva-IN ; | sd-Deva ; | sd-IN
sd-IN ; | sd-Deva-IN ; | sd-Deva ; | sd-IN
yue-CN ; | yue-Hans-CN ; | yue-Hans ; | yue-CN
yue-Hans ; | yue-Hans-CN ; | yue-Hans ; | yue-CN
und-Hant ; | zh-Hant-TW ; | zh-Hant ; | zh-TW
und-Hant-TW ; | zh-Hant-TW ; | zh-Hant ; | zh-TW
und-TW ; | zh-Hant-TW ; | zh-Hant ; | zh-TW
zh-Hant ; | zh-Hant-TW ; | zh-Hant ; | zh-TW
zh-TW ; | zh-Hant-TW ; | zh-Hant ; | zh-TW
—
Reply to this email directly, view it on GitHub
<#3173 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAIXXIUTRX2NAXWD5NDCDNTXUV4GXANCNFSM6AAAAAA3MEDNDA>
.
You are receiving this because your review was requested.Message ID:
***@***.***>
|
und-AX ; sv-Latn-AX ; sv-AX ; | ||
und-AZ ; az-Latn-AZ ; az ; | ||
und-Adlm ; ff-Adlm-GN ; ff-Adlm ; | ||
und-Adlm-BF ; fr-Adlm-BF ; ; |
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.
Could you please explain what does the RemoveFavorRegion = "" mean?
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.
It's in the header. In other words, if a value is missing, it means the same as the previous field in the row (recursively).
# RemoveFavorScript: Remove Likely Subtags, when the script is favored.
# Only included when different than AddLikely.
# RemoveFavorRegion: Remove Likely Subtags, when the region is favored.
# Only included when different than RemoveFavorScript.
If that isn't clear we can improve the header.
CLDR-16970
It became clear in today's ICU meeting that the lack of a test file has led to incompatibilities in implementations. I made a quick generator for the test file. Please check the results to see if there are any unexpected values.
ALLOW_MANY_COMMITS=true