Skip to content

Added BGNPCGN Dzongkha system 2010 - #26

Closed
manuelfuenmayor wants to merge 2 commits into
mainfrom
romanization-Dzongkha
Closed

Added BGNPCGN Dzongkha system 2010#26
manuelfuenmayor wants to merge 2 commits into
mainfrom
romanization-Dzongkha

Conversation

@manuelfuenmayor

@manuelfuenmayormanuelfuenmayor commented Nov 24, 2019

Copy link
Copy Markdown
Contributor

I used the following chart to find relevant unicode values:
https://www.unicode.org/charts/PDF/U0F00.pdf

And the following link to convert them into characters:
https://www.branah.com/unicode-converter

@ronaldtse

Copy link
Copy Markdown
Contributor

Currently failing on these tests:

16)Interscriptmaps/bgnpcgn-dzo-Tibt-Latn-2010.yamlsystemtestfor{"source"=>"ཐྀམ་ཕུ","expected"=>"Thimphu"}Failure/Error: expect(result).toeq(test["expected"])expected: "Thimphu"got: "thaྀm་phaུ"(comparedusing ==)# ./spec/interscript_spec.rb:13:in `block (5 levels) in <top (required)>'17)Interscriptmaps/bgnpcgn-dzo-Tibt-Latn-2010.yamlsystemtestfor{"source"=>"བཀྲ་ཤིས་གྱང་ཙེ","expected"=>"Trashi Yangtse"}Failure/Error: output[pos,match.size]=resultTypeError:
noimplicitconversionofArrayintoString# ./lib/interscript.rb:48:in `[]='# ./lib/interscript.rb:48:in `block in transliterate'# ./lib/interscript.rb:44:in `each'# ./lib/interscript.rb:44:in `transliterate'# ./spec/interscript_spec.rb:12:in `block (5 levels) in <top (required)>'18)Interscriptmaps/bgnpcgn-dzo-Tibt-Latn-2010.yamlsystemtestfor{"source"=>"སྟང་སི་སྦྱིས","expected"=>"Tangsibji"}Failure/Error: expect(result).toeq(test["expected"])expected: "Tangsibji"got: "ྟng་ི་ྦྱི"(comparedusing ==)# ./spec/interscript_spec.rb:13:in `block (5 levels) in <top (required)>'

@manuelfuenmayor

Copy link
Copy Markdown
ContributorAuthor

I think it is worth to say that I took these tests from the notes section of the PDF file. For example:

capture5

@ronaldtse

Copy link
Copy Markdown
Contributor

Replaced by interscript/maps#166

Sign up for freeto 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.

2 participants

@manuelfuenmayor@ronaldtse