summaryrefslogtreecommitdiff
path: root/decoder
diff options
context:
space:
mode:
authorgraehl <graehl@ec762483-ff6d-05da-a07a-a48fb63a330f>2010-07-23 22:44:20 +0000
committergraehl <graehl@ec762483-ff6d-05da-a07a-a48fb63a330f>2010-07-23 22:44:20 +0000
commita4187c028432c9f9ec3693747abc1c52ef310deb (patch)
tree605700d0c6a9cb8ed002c8e7d3b2dc7f282217cc /decoder
parentaeb5ce46a166a4c405f8928bfe39e898f42489d7 (diff)
comment about unary (1 var) goal rule having no target terminals - seems to be assumed
git-svn-id: https://ws10smt.googlecode.com/svn/trunk@392 ec762483-ff6d-05da-a07a-a48fb63a330f
Diffstat (limited to 'decoder')
-rw-r--r--decoder/ff.h1
-rw-r--r--decoder/ff_lm.cc1
2 files changed, 2 insertions, 0 deletions
diff --git a/decoder/ff.h b/decoder/ff.h
index a0b39c26..9ff67dd8 100644
--- a/decoder/ff.h
+++ b/decoder/ff.h
@@ -184,6 +184,7 @@ class ModelSet {
std::string* residual_context,
prob_t* combination_cost_estimate = NULL) const;
+ //this is called INSTEAD of above when result of edge is goal (must be a unary rule - i.e. one variable, but typically it's assumed that there are no target terminals either (e.g. for LM))
void AddFinalFeatures(const std::string& residual_context,
Hypergraph::Edge* edge,
SentenceMetadata const& smeta) const;
diff --git a/decoder/ff_lm.cc b/decoder/ff_lm.cc
index 28312b4f..6579fbee 100644
--- a/decoder/ff_lm.cc
+++ b/decoder/ff_lm.cc
@@ -303,6 +303,7 @@ class LanguageModelImpl {
return ProbNoRemnant(len - 1, len);
}
+ //FIXME: this assumes no target words on final unary -> goal rule. is that ok?
// for <s> (n-1 left words) and (n-1 right words) </s>
double FinalTraversalCost(const void* state) {
if (unigram) return 0.;