Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Parser output on shell is different from the demo output #24

Open
paridhij opened this issue Mar 17, 2016 · 6 comments
Open

Parser output on shell is different from the demo output #24

paridhij opened this issue Mar 17, 2016 · 6 comments

Comments

@paridhij
Copy link

I tried a sentence "DES reduces the risk of recurrent MI, CV death and all-cause mortality" on the TurboParser online demo and the downloaded TurboParser2.3.0. I am using parse.sh to get the conll format of the parse tree. Here are the outputs:

Demo output [which is correct]:
1 DES _ NN NNP _ 2 nsubj _ _
2 reduces _ VB VBZ _ 0 null _ _
3 the _ DT DT _ 4 det _ _
4 risk _ NN NN _ 2 dobj _ _
5 of _ IN IN _ 4 prep _ _
6 recurrent _ JJ JJ _ 7 amod _ _
7 MI _ NN NNP _ 5 pobj _ _
8 , _ , , _ 7 punct _ _
9 CV _ NN NNP _ 10 nn _ _
10 death _ NN NN _ 7 conj _ _
11 and _ CC CC _ 7 cc _ _

12 all-cause _ JJ JJ _ 13 amod _ _
13 mortality _ NN NN _ 7 conj _ _

while the TurboParser2.3.0 installed version gives:

1 DES _ NNP NNP _ 2 SUB
2 reduces _ VBZ VBZ _ 0 ROOT
3 the _ DT DT _ 4 NMOD
4 risk _ NN NN _ 2 OBJ
5 of _ IN IN _ 4 NMOD
6 recurrent _ JJ JJ _ 7 NMOD
7 MI _ NNP NNP _ 13 NMOD
8 , _ , , _ 13 P
9 CV _ NNP NNP _ 13 NMOD
10 death _ NN NN _ 13 NMOD
11 and _ CC CC _ 13 NMOD
12 all-cause _ NN NN _ 13 NMOD
13 mortality _ NN NN _ 5 PMOD

Lines 8,9,10,11 have wrong dependencies, and node 13 has incorrect head. What am I missing here.

@paridhij
Copy link
Author

I also tried with all the tree models, basic, full and standard. None of the models give the results demo gives.

@paridhij
Copy link
Author

Can you please look into the discrepancy?

@andre-martins
Copy link
Owner

Hi,

Which demo are you refering to? http://demo.ark.cs.cmu.edu/parse?

The reason for the discrepancy is that you are using two different models.
Note that the dependency labels are different (Stanford dependencies in the
first example, in the demo model; Yamada-Matsumoto dependencies in the
second one, your local model). To get the same dependencies as the demo,
please contact Lingpeng Kong (CC'ed) who may be able to point you to the
model being used at the demo.

Thanks,

André

2016-03-17 6:06 GMT+00:00 paridhij [email protected]:

I tried a sentence "DES reduces the risk of recurrent MI, CV death and
all-cause mortality" on the TurboParser online demo and the downloaded
TurboParser2.3.0. I am using parse.sh to get the conll format of the parse
tree. Here are the outputs:

Demo output [which is correct]:
1 DES _ NN NNP _ 2 nsubj _ _
2 reduces _ VB VBZ _ 0 null _ _
3 the _ DT DT _ 4 det _ _
4 risk _ NN NN _ 2 dobj _ _
5 of _ IN IN _ 4 prep _ _
6 recurrent _ JJ JJ _ 7 amod _ _
7 MI _ NN NNP _ 5 pobj _ _
8 , _ , , _ 7 punct _ _
9

_CV _ NN NNP _ 10 nn _ _ 10 death _ NN NN _ 7 conj _ _ 11 and _ CC CC _ 7
cc _ __
12 all-cause _ JJ JJ _ 13 amod _ _
13 mortality _ NN NN _ 7 conj _ _

while the TurboParser2.3.0 installed version gives:

1 DES _ NNP NNP _ 2 SUB
2 reduces _ VBZ VBZ _ 0 ROOT
3 the _ DT DT _ 4 NMOD
4 risk _ NN NN _ 2 OBJ
5 of _ IN IN _ 4 NMOD
6 recurrent _ JJ JJ _ 7 NMOD
7 MI _ NNP NNP _ 13 NMOD
8 , _ , , _ 13 P
9 CV _ NNP NNP _ 13 NMOD
10 death _ NN NN _ 13 NMOD
11 and _ CC CC _ 13 NMOD
12 all-cause _ NN NN _ 13 NMOD
13 mortality _ NN NN _ 5 PMOD

Lines 8,9,10,11 have wrong dependencies, and node 13 has incorrect head.
What am I missing here.


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#24

@paridhij
Copy link
Author

Thanks Andre.

@LittleYUYU
Copy link

LittleYUYU commented May 15, 2016

Is the problem solved? Actually I also want the Turbo parser being used in the demo. Could you also put this version online? Thanks much!

(Update: Already got the demo version. Thx!)

@qili5
Copy link

qili5 commented May 25, 2018

I am also very interested to get the Turbo parser used in the demo.
http://demo.ark.cs.cmu.edu/parse
This is so far the most accurate parser for my corpus.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants