Hi all,

One of the latest commits (the one reintroducing alignment info to the
phrasetable) shuffles the info in the phrasetable around.

Here a quick comparison:

commit 3357: 
! ||| ! ||| 0.863961 0.642857 0.604773 0.529412 2.718

commit 3362:
! ||| ! ||| 0.604773 0.529412 ||| 0.863961 0.642857 10 2.718

Unsurprisingly, the decoder can't handle the new format and crashes.

I don't know if this change is intended and you're planning to adjust the
decoder, or if you consider this a bug that you're going to revert, but I
strongly plead for keeping the phrasetables backward compatible. I can't think
of any reason to shuffle the probabilities around that justifies breaking
compatibility, with all the disadvantages this entails.

best regards,
Rico

_______________________________________________
Moses-support mailing list
[email protected]
http://mailman.mit.edu/mailman/listinfo/moses-support

Reply via email to