cldr icon indicating copy to clipboard operation
cldr copied to clipboard

CLDR-8386 fix Malayalam transliterator

Open mjansche opened this issue 4 years ago • 14 comments

Fix transliteration of Malayalam VOWEL SIGN O, OO, AU. Enable transliteration of Malayalam chillu letters. As a result, the transliteration of ജോർദ്ദാൻ into Latin changes from "jēāർddāൻ" to "jōrddān".

Associated changes:

  • Map Dravidian RRA to Bengali RA instead of RA NUKTA.

CLDR-8386

  • [x] This PR completes the ticket.

mjansche avatar Oct 03 '21 16:10 mjansche

FYI @iwsfutcmd

srl295 avatar Oct 04 '21 17:10 srl295

Note that this will go into v41

srl295 avatar Oct 04 '21 17:10 srl295

That makes sense to me.

Mark

On Mon, Oct 4, 2021 at 4:17 PM Martin Jansche @.***> wrote:

@.**** commented on this pull request.

In common/transforms/Malayalam-InterIndic.xml https://github.com/unicode-org/cldr/pull/1515#discussion_r721779764:

@@ -87,6 +106,15 @@ For terms of use, see http://www.unicode.org/copyright.html

൭→\uE06D; # DIGIT SEVEN

൮→\uE06E; # DIGIT EIGHT

൯→\uE06F; # DIGIT NINE

+൰→\uE067\uE066; # UNMAPPED Malayalam-InterIndic: NUMBER TEN

+൱→\uE067\uE066\uE066; # UNMAPPED Malayalam-InterIndic: NUMBER ONE HUNDRED

+൲→\uE067\uE066\uE066\uE066;# UNMAPPED Malayalam-InterIndic: NUMBER ONE THOUSAND

+ൺ→\uE023\uE04D; # LETTER CHILLU NN, remapped to NNA VIRAMA

+ൻ→\uE028\uE04D; # LETTER CHILLU N, remapped to NA VIRAMA

Would it make sense to separately add a set of transform rules that map the <5.1 chillu characters to their >=5.1 equivalents?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/unicode-org/cldr/pull/1515#discussion_r721779764, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACJLEMHJ6JKCD73TXOKQ55TUFIYYPANCNFSM5FHYWMZQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

macchiati avatar Oct 05 '21 01:10 macchiati

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot

I have no idea why this would fail: https://github.com/unicode-org/cldr/actions/runs/1518165589

I've replicated the test setup as closely as I can locally but cannot reproduce the failure. The error messages are consistent with the changed transform not being loaded, but I see no indication that this is actually the case.

mjansche avatar Dec 02 '21 01:12 mjansche

Tenga en cuenta que esto entrará en v41

cesargerardoromo avatar Dec 02 '21 01:12 cesargerardoromo

Hooray! The files in the branch are the same across the force-push. 😃

~ Your Friendly Jira-GitHub PR Checker Bot