thx. I've checked it in and added regression test, so hopefully it won't happen again
On 22 November 2012 14:06, Rico Sennrich <[email protected]> wrote: > On Thu, 2012-11-22 at 14:01 +0000, Hieu Hoang wrote: > > i'll take a look at it straight away. It would be a slip-up on our > > part if it did happen. > > > > From the commit log, it may have happened on the 26/9 or 18/8. > > > > > https://github.com/moses-smt/mosesdecoder/commits/master/phrase-extract/consolidate.cpp > > > > of the > > extract > > score > > consolidate > > programs that makes up the training, only the consolidate doesn't have > > regression tests. I'm gonna make 1 asap > > I think lines 275 and 280 in consolidate-main.cpp should be flipped. > Haven't tested this yet. > >
_______________________________________________ Moses-support mailing list [email protected] http://mailman.mit.edu/mailman/listinfo/moses-support
