cogcomp-nlp

cogcomp-nlp copied to clipboard

Edison search interface.

2

Edison search interface: No Found!

rodriguesfas

Error "Attempted to access negative index of FVector.!

4

I'm tried the provided example in the cogcomp-nlp readme and a got a wierd Error that could not figure out how to solve it. Please me. My code : `String...

PhdStudent2015

duplicate call for two supposedly different scores: https://github.com/CogComp/cogcomp-nlp/blob/master/similarity/src/main/java/edu/illinois/cs/cogcomp/wsim/wordnet/PathFinder.java#L163 presumably, supposed to consider both directions for the relationship by default (i.e. rel(w1, w2) and rel(w2, w1) which makes sense if the...

mssammon

Error in parsing Stanford outputs

5

For dependency parse of the sentence: > Comedian-actress Roseanne Barr sang her famous shrieking rendition of the "Star Spangled Banner" before a San Diego Padres Cincinnati Reds game. Our demo...

nitishgupta

TL;DR: We will need to retrain PrepSRL, Quantifier, (maybe) Question Typer The [Fix](https://github.com/CogComp/cogcomp-nlp/commit/0db5962ba071db7d8438aa958abf411e732c2218#diff-2eacf31fd2ccc45f721ab67ab2db70acR62) applied to #593 (currently residing in branch [string_join_require_retrain](https://github.com/CogComp/cogcomp-nlp/tree/string_join_require_retrain)) will affect the behavior of [FeatureNGramUtility ](https://github.com/CogComp/cogcomp-nlp/blob/7d9dad3fedc16ac59feb278815e27dc195d1367e/edison/src/main/java/edu/illinois/cs/cogcomp/edison/features/helpers/FeatureNGramUtility.java) in edison....

schen149

String.join includes an extra copy of separator at the end of the string.

3

https://github.com/CogComp/cogcomp-nlp/blob/ce0f3a03264d293cd751d7a416bbadd858066496/core-utilities/src/main/java/edu/illinois/cs/cogcomp/core/utilities/StringUtils.java#L60 There is a .trim() call, but this only trims whitespace. If the separator is non-whitespace (e.g. underscore, dash, period, bar, etc.), then trim doesn't work. Also, Apache Commons has...

mayhewsw

no helpful error message if model file not found

2

no indication that minio looked anywhere -- am I wrong? just an error about path it tried that did not work. Looks like there is now a disconnect between the...

mssammon

Issue 665 fix a bug which allowed the overlapping spans check in SpanLabelView to be gotten around by using SpanLabelView.addConstituent(Constituent). This fix broke some code because there were instances of...

ChaseDuncan

SpanLabelView may not enforce non-overlap constraint

4

It looks like if you call `addConstituent()` it sidesteps the span check: https://github.com/CogComp/cogcomp-nlp/blob/master/core-utilities/src/main/java/edu/illinois/cs/cogcomp/core/datastructures/textannotation/SpanLabelView.java#L62 This should perform a bounds check unless there is a good reason not to; otherwise, documentation should...

mssammon

WIP: At the request of #593

1

Try to solve #593 Remove join function in utilities/StringUtils.java, use StringUtils.join in Apache common

dannima