cdec
cdec copied to clipboard
redpony
Decoder, aligner, and model optimizer for statistical machine translation and other structured prediction models based on (mostly) context-free formalisms
In functions `ComputeZ` and `ComputeDLogZ`, [line 33 and 50](https://github.com/redpony/cdec/blob/master/word-aligner/da.h#L33): ``` C++ const unsigned num_top = n - floor; ``` `floor` may be greater than `n`, which makes `num_top` reach values...
During the step of compiling the bilexical dictionary (cdec/python/cdec/sa/compile.py line 124) I get a Segmentation Fault after some time. INFO:cdec.sa.compile:Compiling source suffix array INFO:cdec.sa.compile:Compiling source suffix array took ... seconds...
When computing emp_feat in fast_align.cc(line 205), I think index j should be j+1 since j start from 0. emp_feat += DiagonalAlignment::Feature(j, i, trg.size(), src.size()) \* p; The other parts pass...
Hi, I am not sure if this problem existed in earlier versions, but at least for the current version there is a problem when translation input contains non-terminal-like symbols, such...
I would like to point out that identifiers like "[`_ALIGNER_H_`](https://github.com/redpony/cdec/blob/b8f314dddda3d440164e4772830e3c951ba06ee4/decoder/aligner.h#L1)" and "[`_HG_H_`](https://github.com/redpony/cdec/blob/d033a045aa46ff876ad2c9f6929e2095b2481cdf/decoder/hg.h#L1)" [do not fit](https://www.securecoding.cert.org/confluence/display/cplusplus/DCL32-CPP.+Do+not+declare+or+define+a+reserved+identifier#DCL32-CPP.Donotdeclareordefineareservedidentifier-NoncompliantCodeExample%28HeaderGuard%29) to the expected naming convention of the C++ language standard. Would you like to adjust...
During python install, after > python setup.py build 1)user should also execute, as root, > python setup.py install 2)PYTHONPATH should be checked, or at least set to properly fetch modules...
I wanted to do a fair comparison with cdec since my code runs much faster with the lower order estimates. You wanted this in code review form.
http://cdec-decoder.org/index.php?title=Building_cdec_from_source doesn't mention that flex is a requirement
The parser does not always apply unary rules in the right order (in particular, when rules are loaded from multiple grammars). This can cause parse failures and missing derivations. Example:...
Hi, In the makefiles of the latest cdec, '-mt' suffix are appended for libboost. It is a bit inconvenient for linux users since default installation on linux does not have...
