Re: new module ctakes-scaleout is missing [EXTERNAL]

2017-09-27 Thread Alexandru Zbarcea
Hi Sean, Reverting the pom.xml did the trick. While doing the changes to the pom.xml, I wonder if my fix for CTAKES-457[1] breaks your current setup. Is there anything you recommend me to add in order to make the patch ready to be applied? Thanks, Alex Zbarcea [1] -

[CTAKES-450] CDA Segment Annotator misses all headings after empty segment

2017-09-27 Thread Alexandru Zbarcea
Hi, I have provided another patch for CTAKES-450[1]. The fix provided in the original patch is correct. However, I tried to address also small issues and some refactoring, especially addressing the Unit Testing. If you think it requires more explanation, let me know. * extracted methods *

RE: new module ctakes-scaleout is missing [EXTERNAL]

2017-09-27 Thread Finan, Sean
Argh. Apparently that was my fault - I have a sandbox scaleout project, but the change to the main pom is supposed to be in a separate changelist. Somehow I managed to check it in. That is what I get for having a dozen unfinished side projects ... I have reverted the main pom back to its

new module ctakes-scaleout is missing

2017-09-27 Thread Alexandru Zbarcea
Hi, Today, trying to build latest changes to ctakes, I noticed that a new dependency over ctakes-scaleout module has been introduced, but I couldn't find this module. [ERROR] [ERROR] Some problems were encountered while processing the POMs: [ERROR] Child module ctakes/ctakes-scaleout of

RE: Enabling drugner pipeline and identifying dates [EXTERNAL] [SUSPICIOUS]

2017-09-27 Thread Finan, Sean
Hi Gandhi, I am glad that you are feeling better. I don't understand why you aren't getting the same output as me. I just ran your example sentence with your piper with a fresh checkout and get the html below. The css follows. Copy and paste into a file and see if you see the corefs.

RE: Enabling drugner pipeline and identifying dates [EXTERNAL] [SUSPICIOUS]

2017-09-27 Thread Gandhi Rajan Natarajan
Hi Sean, Sorry for the delayed response as I was out of office due to illness. If I don't add BackwardsTimeAnnotator, I don't see any error related to isTraining param. But still couldn't get the superscript co-reference working. Please note that I am using the latest 4.0.1 jars. The piper