That looks suspicious. Did you look at the history?
Offhand I think we would want to skip those with NE_TYPE_ID_UNKNOWN all 
together, and maybe log a message.

-- James


> -----Original Message-----
> From: [email protected] [mailto:dev-
> [email protected]] On Behalf Of Chen,
> Pei
> Sent: Monday, April 22, 2013 3:20 PM
> To: [email protected]
> Subject: NamedEntityLookupConsumerImpl
> 
> Should we be defaulting UNKNOWN type NE's to MedicationEventMention?
> Wouldn't it make more sense if CONST.NE_TYPE_ID_UNKNOWN to create an
> EventMention rather than Medication?
> 
> <snip>
>          IdentifiedAnnotation neAnnot;
>          if ( typeId == CONST.NE_TYPE_ID_DRUG || typeId ==
> CONST.NE_TYPE_ID_UNKNOWN ) {
>             neAnnot = new MedicationEventMention( jcas );
>          } else {
>             neAnnot = new EntityMention( jcas );
>          }
> </snip>

Reply via email to