Difference between revisions of "User:SilentFlame/Progress"

From Apertium
Jump to navigation Jump to search
Line 5: Line 5:
 
# Apply changes to transfer.cc to interchunk.cc
 
# Apply changes to transfer.cc to interchunk.cc
 
#* Check <code>git clone -b blank-handling https://github.com/unhammer/apertium</code>
 
#* Check <code>git clone -b blank-handling https://github.com/unhammer/apertium</code>
#* Apply the diff (between that branch and master) from transfer.cc to interchunk.cc
+
#* Apply the diff 4c7c4f8f1b..2025182991 from transfer.cc to interchunk.cc
 
#* Try to make it compile and run – report things that didn't seem to have a 1-1 correspondence
 
#* Try to make it compile and run – report things that didn't seem to have a 1-1 correspondence
 
#* Write tests for interchunk, like those for transfer at https://github.com/unhammer/apertium/tree/blank-handling/tests
 
#* Write tests for interchunk, like those for transfer at https://github.com/unhammer/apertium/tree/blank-handling/tests

Revision as of 09:23, 30 May 2017

Progress on Automatic_blank_handling

Current task

Interchunk

  1. Apply changes to transfer.cc to interchunk.cc

TODO

Deformatters

Reformatters

lttoolbox

hfst

transfer (non-chunking)

  • Test if current transfer.cc handles non-chunking/single-stage transfer correctly, if not, fix
  • Task: PR to https://github.com/unhammer/apertium/ with tests showing working transfer.cc for single-stage/non-chunking transfer, with inline vs block-level blank handling and test that rules using misnumbered/missing b-elements should not mess up formatting

postchunk

(Should be done after interchunk is complete)

  • Task: PR to https://github.com/unhammer/apertium/ including tests showing working postchunk blank handling – test that rules using misnumbered/missing b-elements should not mess up formatting

etc

  • Ensure all other modules are fine with the new format for inline blanks (e.g. cg-proc)
  • Work on other deformatters (mediawiki? latex?)

Done

(Some of these are from coding challenges)

deformatting prototypes

  1. Make the HTML format handler apertium-deshtml turn "<i>foo <b>bar</b></i>" into "[{<i>}]foo [{<i><b>}]bar"

pretransfer

transfer (chunker)

  1. Fix a memory bug
    • uncommenting apertium/transfer.cc:1259 // delete[] format; in the blank handling branch leads to a double-free – find out why and ensure we're correctly releasing memory
    • Install valgrind from your package manager or http://valgrind.org/, then compile your program with -O0 -g3, then run valgrind -v --leak-check=full apertium/apertium-transfer and read the output