search

hltfbk / Excitement-Transduction-Layer

1 stars 1 forks source link

CasTreeConverter exceptions for NICE data with BIUTEE #185

Closed LiliKotlerman closed 10 years ago

LiliKotlerman commented 10 years ago

BIUTEE EDA crashes on NICE data with an exception I cannot fully understand (see below).

The problem is that to reproduce the error one has to install and be able to run BIUTEE. It's not difficult, I listed the steps here https://github.com/hltfbk/Excitement-Transduction-Layer/issues/178 Yet, maybe someone can understand what causes this exception without seeing the error in action?

When I'm running on WP2_public_data_CAS_XMI/nice_email_3 with file limit of 15 the run goes successfully. When I take more files, say, 25 - it falls. So I guess the problem is somewhere in our inputs... Additional strange thing is that all the files from WP2_public_data_CAS_XMI/nice_email_3 are processed by TIE (default demo params) with no errors.

148019 [main] INFO eu.excitementproject.eop.biutee.rteflow.systems.excitement.BiuteeEDA - Building PairData from the given JCas... Exception in thread "main" java.lang.NullPointerException at java.util.ArrayList.(ArrayList.java:151) at eu.excitementproject.eop.lap.biu.uima.CasTreeConverter.addChildren(CasTreeConverter.java:297) at eu.excitementproject.eop.lap.biu.uima.CasTreeConverter.convertSentenceToTree(CasTreeConverter.java:188) at eu.excitementproject.eop.biutee.rteflow.systems.excitement.CasPairDataConverter.convertCasToPairData(CasPairDataConverter.java:71) at eu.excitementproject.eop.biutee.rteflow.systems.excitement.BiuteeEdaUtilities.convertJCasToPairData(BiuteeEdaUtilities.java:51) at eu.excitementproject.eop.biutee.rteflow.systems.excitement.BiuteeEDA.process(BiuteeEDA.java:99) at eu.excitementproject.tl.structures.rawgraph.EntailmentRelation.computeTEdecision(EntailmentRelation.java:176) at eu.excitementproject.tl.structures.rawgraph.EntailmentRelation.(EntailmentRelation.java:75) ...

gilnoh commented 10 years ago

Hello, Lili.

CasTreeConverter is part of BIUTEE. I think it requires some assumption on the input, but the input wasn't satisfying this -- or rather, a bug. You have to consult Ofer, who wrote those codes. (The code is part of BIUTEE LAP, that converts parse tree of CAS into BIUTEE (now EOP) internal BasicNode tree.

And this is only used by BIUTEE, and that's the reason why only BIUTEE makes this exception. (e.g. TIE uses CAS tree itself directly, with no conversion).

LiliKotlerman commented 10 years ago

Gil, thanks! You did save me a lot of debugging. Since Ofer is not really available now, I guess for now I'll add some patch to my code to exclude erroneous fragments from the process... And I'll inform the BIUTEE team, of course.