Skip to content

CLDR-17885 Sunuwar - add suz, suz_Deva, suz_Sunu#4651

Merged
srl295 merged 7 commits intounicode-org:mainfrom
codehivetx:CLDR-17885/suz
May 7, 2025
Merged

CLDR-17885 Sunuwar - add suz, suz_Deva, suz_Sunu#4651
srl295 merged 7 commits intounicode-org:mainfrom
codehivetx:CLDR-17885/suz

Conversation

@srl295
Copy link
Member

@srl295 srl295 commented Apr 30, 2025

CLDR-17855

includes font from #4214 @NeilSureshPatel

  • This PR completes the ticket.

ALLOW_MANY_COMMITS=true
DISABLE_JIRA_ISSUE_MATCH=true

@srl295 srl295 self-assigned this Apr 30, 2025
@jira-pull-request-webhook
Copy link

Notice: the branch changed across the force-push!

  • common/supplemental/likelySubtags.xml is now changed in the branch
  • common/supplemental/supplementalData.xml is now changed in the branch
  • common/supplemental/supplementalMetadata.xml is different
  • tools/cldr-code/src/main/resources/org/unicode/cldr/util/data/country_language_population.tsv is now changed in the branch
  • tools/cldr-code/src/main/resources/org/unicode/cldr/util/data/language_script.tsv is different
  • tools/cldr-code/src/main/resources/org/unicode/cldr/util/data/locales/scriptToExemplars.txt is now changed in the branch

View Diff Across Force-Push

~ Your Friendly Jira-GitHub PR Checker Bot

@srl295
Copy link
Member Author

srl295 commented Apr 30, 2025

hey @conradarcturus i. take back all complaints about generating data too early! Even the ones i didn't remember to send.

@DavidLRowe
Copy link
Contributor

@srl295 In supplementalData.xml, I would have expected parentLocale="root" to have "suz_Sunu" rather than "suz_Deva". AFAICT suz-Deva = suz, but suz-Sunu needs to fall back to root, not to suz, thus suz_Sunu needs to be listed here (not suz_Deva).

@DavidLRowe
Copy link
Contributor

In the log, there's an error :

Error: (LikelySubtagsTest.java:574) Error: : Deva: expected "[ँ-ःअ-ऍए-ऑओ-नपफभ-रलळव-ह़-ॅे-ॉो-्ॐ-॒ॠ-ॣॻॼॾॿ‌‍𑯚]", got "[ँ-ःअ-ऍए-ऑओऔखघङछझञठढणथधफभळशष़ऽिू-ॅैॉौ्ॐ-॒ॠ-ॣॻॼॾॿ‌‍]"

I don't know where that error is generated or what to do to fix it. AFAICT the second set of characters is a subset of the first. The first uses a number of ranges and includes more characters than the second.

It seems there is a Sunuwar character (U+11BDA Sunuwar letter ngar) just before the closing square bracket in the "expected" string, which seems odd.

@srl295
Copy link
Member Author

srl295 commented May 1, 2025

In the log, there's an error :

Error: (LikelySubtagsTest.java:574) Error: : Deva: expected "[ँ-ःअ-ऍए-ऑओ-नपफभ-रलळव-ह़-ॅे-ॉो-्ॐ-॒ॠ-ॣॻॼॾॿ‌‍𑯚]", got "[ँ-ःअ-ऍए-ऑओऔखघङछझञठढणथधफभळशष़ऽिू-ॅैॉौ्ॐ-॒ॠ-ॣॻॼॾॿ‌‍]"

I don't know where that error is generated or what to do to fix it. AFAICT the second set of characters is a subset of the first. The first uses a number of ranges and includes more characters than the second.

It seems there is a Sunuwar character (U+11BDA Sunuwar letter ngar) just before the closing square bracket in the "expected" string, which seems odd.

yeah, there's an untranslated sunuwar char, i will check that.

It's "expected" because it expected to find that etter in the static list but didn't.

@NeilSureshPatel
Copy link

I just took look at the Sunuwar-InterIndic_SWS transform and the ngar is defined in the setup but it is missing in the transforms. I am guessing it is just passing through unchanged.

@srl295
Copy link
Member Author

srl295 commented May 1, 2025

I just took look at the Sunuwar-InterIndic_SWS transform and the ngar is defined in the setup but it is missing in the transforms. I am guessing it is just passing through unchanged.

OK. I will fix it.

I had some issues with the transforms, that's why I made a new PR that has heavy hand editing. Part might be in the transforms, btu there are also serious CLDR/ICU tooling issues.

@srl295
Copy link
Member Author

srl295 commented May 7, 2025

I just took look at the Sunuwar-InterIndic_SWS transform and the ngar is defined in the setup but it is missing in the transforms. I am guessing it is just passing through unchanged.

the test file https://github.com/unicode-org/cldr/pull/4214/files#diff-1684eb9afa19fb9944dc3bf883a450979eff9787c04ac8ffd506c835b359322dR29 shows it unchanged also

srl295 added 3 commits May 7, 2025 14:42
- fix exemplar errors in suz(Deva) and suz_Sunu
- update attributeValueValidity
@srl295 srl295 requested review from DavidLRowe, btangmu, conradarcturus, macchiati and pedberg-icu and removed request for btangmu and conradarcturus May 7, 2025 21:31
@DavidLRowe
Copy link
Contributor

See my previous comment about suz_Deva in parentLocale parent="root" in supplementalData.xml. I think it should be suz_Sunu (or else I'd like to know why I'm wrong, which is entirely possible!).

@srl295 srl295 merged commit da55a43 into unicode-org:main May 7, 2025
9 checks passed
@srl295 srl295 deleted the CLDR-17885/suz branch May 7, 2025 23:00
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.

3 participants