search

clulab / eidos

Machine reading system for World Modelers
Apache License 2.0
37 stars 24 forks source link

P4S2 in TestCagP2.scala #82

Closed Fan-Luo closed 6 years ago

Fan-Luo commented 6 years ago

Sentence:

Violence has caused livestock to be looted, killed and disease-prone and crops destroyed, and displacement has caused delayed planting.

Test:

tester.test(newEdgeSpec(displacement, Causal, delayedPlanting)) should be (successful)

Problem: 

delayedPlanting is not extracted, as shown in figure below.

I tried to debug and noticed that both ‘delayed’ and ‘planting’ are considered as I-VP

The tag for 'delayed planting’ also seems wrong: ‘delayed’ is considered as VBN, and ‘planting’ is considered as VBG.

This issue seems from tagPartsOfSpeech in processor.

MihaiSurdeanu commented 6 years ago

I think we should give up on this. That is, change delayed planting to another NP that is correctly recognized.

On Feb 15, 2018, at 5:35 PM, Fan-Luo notifications@github.com wrote:

Sentence:

Violence has caused livestock to be looted, killed and disease-prone and crops destroyed, and displacement has caused delayed planting.

Test:

tester.test(newEdgeSpec(displacement, Causal, delayedPlanting)) should be (successful)

Problem:

delayedPlanting is not extracted, as shown in figure below.

I tried to debug and noticed that both ‘delayed’ and ‘planting’ are considered as I-VP

The tag for 'delayed planting’ also seems wrong: ‘delayed’ is considered as VBN, and ‘planting’ is considered as VBG.

This issue seems from tagPartsOfSpeech in processor.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.