Quote Annotation - AnnotationException StringIndexOutOfBoundsException

[allenkim]

Hello,

I had a situation with text that had this: ""=

It seems to throw an error when I try running the pipeline with quote annotation on this small fragment. Just wanted to verify that it was an issue.

Thank you.

[AngledLuffa]

Would you try this? It's not an official release, just a zip of all the recent changes we've made. Note that the current release of the shift-reduce models won't work with this. If you need those, I'll send you a link for that as well. http://downloads.cs.stanford.edu/nlp/software/stanford-corenlp-4.2.1.zip

[allenkim]

I tried running CoreNLP with the new files in the zip, but am still getting the same errors...

[AngledLuffa]

Alright, oops. Would you send us a sample text / program which is causing that crash? I'll try to work it out and post another fix.

[allenkim]

Just running the pipeline on the given text ""= seems to create errors for me when including the quote pipeline. Hopefully, I'm not running something incorrectly.

[AngledLuffa]

Alright, I see a bug and a potential fix using that example. I posted a new release candidate at the same address. Want to try again?

http://downloads.cs.stanford.edu/nlp/software/stanford-corenlp-4.2.1.zip

[allenkim]

Thanks. It seems to work for that example, but here is another example that seems to cause issues: "?'" AAAAA BBBBB

[AngledLuffa]

Try again? I hopefully fixed that crash as well.

[allenkim]

I seem to still be getting the same error for the "?'" AAAAA BBBBB example. Was the same link updated?

[AngledLuffa]

No, it's not. Sorry... fix it in a few minutes

[AngledLuffa]

Alright, I rebuilt it. Try again?

[allenkim]

Seemed to fix that issue, but I found another (sorry to keep bombarding you like this). They all seem to be under this index out of bounds error.

This one is a larger piece of text, so I attached a text file.

excerpt.txt

[AngledLuffa]

This one is not going to have as quick a turnaround, I'm afraid

[AngledLuffa]

Found it, I think. Want to try again, same link?

[AngledLuffa]

PS we hope to make a new release in a week or two, with these changes being part of the updates, so please don't hesitate to send us any more errors you find

[allenkim]

Thanks! This seems to close out the issues I see so far, but I'll keep this open for a bit in case more errors pop up.