That one required a bug fix. Pull and try again, please. - Uli
On Wed, Dec 2, 2015 at 6:39 PM, Jeremy Gwinnup <[email protected]> wrote: > Uli, > > I just tried the suggested settings on a couple systems: > > search algo : > 1 (cube pruning) - yields UNK in both final output and nbest > 3 (CYK+) - yields UNK in final output but not nbest > 5 (inc-search) - yields UNK in final output but not nbest > > I’m guessing since factors aren’t supported in the chart-based algorithms > the bug you mention still happens? > > Thanks again for the help! > -Jeremy > > > > On Dec 2, 2015, at 1:08 PM, Ulrich Germann <[email protected]> > wrote: > > > > Hi Jeremy, > > > > looks like a bug to me at line 1755 in moses/Manager.cpp, when > reportAllFactors is true. > > > > Theoretically, one could have the streaming operator for phrases check > if mark-unknown is set and use the markup, but I'm trying to eliminate the > dependence on global variables in the decoder, so I'm more inclined to have > / use a toString() function that gets the requested behaviour as a function > parameter. > > > > For the time being, a quick hack around the issue might be to set > --report-all-factors to false and set --output-factors to the factors you > want in the output. (If --output-factors is not set, Moses should default > to just printing the first factor, but I haven't tried that out). > > > > - Uli > > > -- Ulrich Germann Senior Researcher School of Informatics University of Edinburgh
_______________________________________________ Moses-support mailing list [email protected] http://mailman.mit.edu/mailman/listinfo/moses-support
