danielmorozoff / bubs-parser

Automatically exported from code.google.com/p/bubs-parser
0 stars 0 forks source link

changeset 1071:c867467ddf39 ignores -beamModel #3

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. checkout RELEASE_1_0 branch, changeset 1035:977a87837605
2. run the following two commands from bash:
a.
echo "The why x w v you tee s are queue pea oh n m el que jay I age gee if e 
thee see b a."  | build-dist/parse -g models/eng.sm6.gr.gz -fom 
models/eng.sm6.fom.gz -beamModel models/eng.sm6.bcm.gz

b.
echo "The why x w v you tee s are queue pea oh n m el que jay I age gee if e 
thee see b a."  | build-dist/parse -g models/eng.sm6.gr.gz -fom 
models/eng.sm6.fom.gz -beamModel models/eng.sm6.bcm.gz -O 
beamModelBias="-200,-200,-200,-200"
3. checkout changeset 1071:c867467ddf39 and compile
4. rerun the commands from (2) and call the output c and d respectively

What is the expected output? What do you see instead?
observed:
a.
(ROOT (SBAR (ADVP (DT The)) (WHADVP (WRB why)) (S (X (SYM x)) (X (FW w) (FW v) 
(NP (PRP you)) (NN tee)) (NP (PRP s)) (VP (VBP are) (S (INTJ (JJ queue) (NN 
pea) (INTJ (UH oh))) (INTJ (JJ n) (NN m) (VB el) (VB que) (VB jay)) (PRN (S (NP 
(PRP I)) (VP (VBP age) (ADVP (JJ gee)) (SBAR (IN if) (X (SYM e)))))) (X (SYM 
thee)) (VP (VBP see) (NP (NN b) (NN a.))))))))
b.
()
c.
(ROOT (S (NP (NP (DT The)) (SBAR (WHADVP (WRB why)) (S (X (SYM x)) (X (FW w) 
(FW v) (NP (PRP you)) (NN tee)) (NP (PRP s)) (VP (VBP are) (NP (JJ queue) (NN 
pea)) (INTJ (UH oh)) (VBP n) (X (FW m) (FW el) (X (FW que) (FW jay) (S (NP (PRP 
I)) (VP (VBP age))) (FW gee)) (SBAR (IN if) (X (SYM e))) (FW thee)))))) (VP 
(VBP see) (NP (NN b) (NN a.)))))
d.
(ROOT (S (NP (NP (DT The)) (SBAR (WHADVP (WRB why)) (S (X (SYM x)) (X (FW w) 
(FW v) (NP (PRP you)) (NN tee)) (NP (PRP s)) (VP (VBP are) (NP (JJ queue) (NN 
pea)) (INTJ (UH oh)) (VBP n) (X (FW m) (FW el) (X (FW que) (FW jay) (S (NP (PRP 
I)) (VP (VBP age))) (FW gee)) (SBAR (IN if) (X (SYM e))) (FW thee)))))) (VP 
(VBP see) (NP (NN b) (NN a.)))))

expected:
a. (same as above)
b. (same as above)
c. (same as a. above)
d. (same as b. above)

What version of the product are you using? On what operating system?
The problem appears in default branch changeset 1071:c867467ddf39 . I'm running 
on 64 bit Linux

Please provide any additional information below.
Lines 349 -- 356 in ParserDriver.java seem suspicious and I can remove the 
above symptom by duplicating line 355 on the line below the current line 350 of 
changeset 1071:c867467ddf39
Running with different -beamModel models in the same version also did not 
result in different behavior as I would have expected.

Original issue reported on code.google.com by adamteic...@gmail.com on 9 Jan 2013 at 9:57

GoogleCodeExporter commented 8 years ago
Fixed in the master branch on 2013-12-29, in revision 6a571f5f729e

Original comment by aaron.du...@gmail.com on 20 Feb 2013 at 5:38