Skip to content

Conversation

@dcalacci
Copy link

  • updated the CR_PATTERN regex to correctly extract the coreference set
    notation
  • updated the jar directory to reflect the most recent release of stanford corenlp

 - updated the CR_PATTERN regex to correctly extract the coreference set
   notation

 - cleaned up the code in the STATE_COREFERENCE if/else branch in
   parse_parser_results to be clearer and actually add the coreference
   results to the result dict.
@dcalacci
Copy link
Author

this is because when I tried to use your code off the shelf I never got any coreference results back in the result dict, but the output in the server console showed results.

so this fixes that problem.

fixed the terrible bug I left in my previous commit
relwell added a commit that referenced this pull request Jan 22, 2014
Fixes coreference results, updates jar directory
@relwell relwell merged commit e671bec into relwell:master Jan 22, 2014
@dcalacci dcalacci deleted the coref branch March 26, 2014 15:42
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

Successfully merging this pull request may close these issues.

2 participants