Hi rico

the format of the phrase table is in flux @ the moment 'cos we've added 
alignment info so the extraction and decoder won't work unless it's all 
same version. we're trying to sort this out.

my suggestion is to update the extraction scripts and decoder to all the 
same version. if you still have problems, let me know. apologies

On 26/07/2010 12:56, Rico Sennrich wrote:
> 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
>
>
_______________________________________________
Moses-support mailing list
[email protected]
http://mailman.mit.edu/mailman/listinfo/moses-support

Reply via email to