[liblouis-liblouisxml] Re: nocontractsign fixes the last regression

  • From: Christian Egli <christian.egli@xxxxxx>
  • To: liblouis-liblouisxml@xxxxxxxxxxxxx
  • Date: Wed, 15 Jun 2016 12:54:22 +0200

Hi Donald

On 06/15/2016 12:34 PM, Donald Winiecki wrote:

Apologies for the question.  Why are we inserting grade 1 symbol
indicators in these cases?  If this is only a test case (as it appears
to be) then the answer is academic, though I am uncertain for what
actual use cases the tests are testing.

Very good question. I inherited these tests from people that know ueb. They worked in liblouis before we integrated the ueb code changes from Mike. So I expect them to still work after the additions from Mike.

But as to whether these are real live use cases or whether they are correct I will have to pass the buck to someone more knowledgeable about ueb.

Thanks
Christian


On Wed, Jun 15, 2016 at 4:13 AM, Christian Egli <christian.egli@xxxxxx
<mailto:christian.egli@xxxxxx>> wrote:

    Hi all

    The last regression in the ueb_update branch is in
    en-ueb-g2-dictionary_harness.txt.bak. These are basically the tests as
    they were before we integrated the ueb changes. They use the old ueb
    tables, so in theory they should behave the same as before. However
    there are some problems such as

    Input:    'AB's'
    Expected: '⠰⠠⠠⠁⠃⠄⠎' (length 7)
    Received: '⠠⠠⠁⠃⠄⠎' (length 6)

    Input:    'AC's'
    Expected: '⠰⠠⠠⠁⠉⠄⠎' (length 7)
    Received: '⠠⠠⠁⠉⠄⠎' (length 6)

    Input:    'Ac'
    Expected: '⠰⠠⠁⠉' (length 4)
    Received: '⠠⠁⠉' (length 3)

    Input:    'Ac's'
    Expected: '⠰⠠⠁⠉⠄⠎' (length 6)
    Received: '⠠⠁⠉⠄⠎' (length 5)

    Input:    'Ag'
    Expected: '⠰⠠⠁⠛' (length 4)
    Received: '⠠⠁⠛' (length 3)

    This could be for two reasons:
    1. We made a mistake when migrating the en-ueb tables to the new
        emphasis opcodes or
    2. there might be a problem with capitalization in the new liblouis
        code.

    I analyzed this some more and found out that if I add the
    `nocontractsign 56` opcode to the old en-ueb the tests pass.

    This opcode is not documented at the moment and I thought it was related
    to numeric mode which is currently disabled. So

    1. what exactly is the nocontractsign opcode and
    2. should I push this fix to the old table

    That would probably indicate that other tables might need this opcode as
    well, since the code seems to behave differently if this opcode isn't
    there.

    Thanks
    Christian

    --
    Christian Egli
    Swiss Library for the Blind, Visually Impaired and Print Disabled
    Grubenstrasse 12, CH-8045 Zürich, Switzerland

    -----
    Tag der offenen Tuer: Es war einmal...
    Die SBS laedt Sie herzlich ein: 25. Juni 2016 von 9 bis 16 Uhr.
    Mehr Informationen erhalten Sie unter www.sbs.ch/offenetuer
    <http://www.sbs.ch/offenetuer>
    For a description of the software, to download it and links to
    project pages go to http://liblouis.org




--
Christian Egli
Swiss Library for the Blind, Visually Impaired and Print Disabled
Grubenstrasse 12, CH-8045 Zürich, Switzerland


-----
Tag der offenen Tuer: Es war einmal...
Die SBS laedt Sie herzlich ein: 25. Juni 2016 von 9 bis 16 Uhr. Mehr Informationen erhalten Sie unter www.sbs.ch/offenetuer
For a description of the software, to download it and links to
project pages go to http://liblouis.org

Other related posts:

  1. Home
  2. liblouis-liblouisxml
  3. Archive
  4. 06-2016