moses-smt / mosesdecoder

Moses, the machine translation system
http://www.statmt.org/moses
GNU Lesser General Public License v2.1
1.58k stars 777 forks source link

EMS: Added missing step for the "TRAINING:build-generation-custom". #198

Closed tofula closed 6 years ago

tofula commented 6 years ago
  1. Added missing step for the "TRAINING:build-generation-custom". Fixed the $cmd parameter - should be "-corpus" instead of "-generation-corpus".
hieuhoang commented 6 years ago

I'm assuming this is correct but I can't be sure. Will pull this in a few days if no-one objects

tofula commented 6 years ago

Working fine for my EMS config. If any test cases can be created, let me know so I can write something...

hieuhoang commented 6 years ago

cheers. There's lots of regression tests but none that tests experiment.perl https://github.com/moses-smt/moses-regression-tests/tree/master/tests It would be a pain to add a new type of test. Lets crossed our fingers and toes and hope nothing is broken

tofula commented 6 years ago

Ok, thanks a lot for the merge!