RE: dictionary-look-fast fails to handle alternative CUIs

2015-07-10 Thread Finan, Sean
completely outside the lookup loop. From: britt fitch [mailto:britt.fi...@wiredinformatics.com] Sent: Friday, July 10, 2015 5:57 PM To: dev@ctakes.apache.org Subject: Re: dictionary-look-fast fails to handle alternative CUIs No issues so far. I think you are already handling the 1 edge case I could

Re: dictionary-look-fast fails to handle alternative CUIs

2015-07-10 Thread britt fitch
reat, thanks. Any issues or concerns? Possible enhancements? Like the > source, I’m open to change … > > From: britt fitch [mailto:britt.fi...@wiredinformatics.com] > Sent: Friday, July 10, 2015 5:29 PM > To: dev@ctakes.apache.org > Subject: Re: dictionary-look-fast fa

RE: dictionary-look-fast fails to handle alternative CUIs

2015-07-10 Thread Finan, Sean
Great, thanks. Any issues or concerns? Possible enhancements? Like the source, I’m open to change … From: britt fitch [mailto:britt.fi...@wiredinformatics.com] Sent: Friday, July 10, 2015 5:29 PM To: dev@ctakes.apache.org Subject: Re: dictionary-look-fast fails to handle alternative CUIs

Re: dictionary-look-fast fails to handle alternative CUIs

2015-07-10 Thread britt fitch
nd close the ticket if it fits your > purposes. > > From: britt fitch [mailto:britt.fi...@wiredinformatics.com] > Sent: Thursday, July 09, 2015 3:30 PM > To: dev@ctakes.apache.org > Subject: Re: dictionary-look-fast fails to handle alternative CUIs > > Linking t

RE: dictionary-look-fast fails to handle alternative CUIs

2015-07-09 Thread Finan, Sean
Checked in, please give it a test and close the ticket if it fits your purposes. From: britt fitch [mailto:britt.fi...@wiredinformatics.com] Sent: Thursday, July 09, 2015 3:30 PM To: dev@ctakes.apache.org Subject: Re: dictionary-look-fast fails to handle alternative CUIs Linking ticket here for

Re: dictionary-look-fast fails to handle alternative CUIs

2015-07-09 Thread britt fitch
Linking ticket here for completeness https://issues.apache.org/jira/browse/CTAKES-368 Britt Fitch Wired Informatics 265 Franklin St Ste 1702 Boston, MA 02110 http://wiredinformatics.com britt.fi...@wiredinformatics.com > On Jul 9, 2015, at 3:1

Re: dictionary-look-fast fails to handle alternative CUIs

2015-07-09 Thread britt fitch
>>> wrote: > > By the way, in case you are wondering why it does this … the umls database > that we use has roughly half a million cuis. Storing cuis in the various > tables as longs takes up a lot less space than storing them as 8 character > strings. > > From: britt

RE: dictionary-look-fast fails to handle alternative CUIs

2015-07-09 Thread Finan, Sean
Hi Britt, I’ve got some code and tests to check in. Would you like to write the jira item? From: britt fitch [mailto:britt.fi...@wiredinformatics.com] Sent: Thursday, July 09, 2015 8:55 AM To: dev@ctakes.apache.org Subject: Re: dictionary-look-fast fails to handle alternative CUIs I don’t

Re: dictionary-look-fast fails to handle alternative CUIs

2015-07-09 Thread britt fitch
uly 08, 2015 4:21 PM > To: dev@ctakes.apache.org <mailto:dev@ctakes.apache.org> > Subject: Re: dictionary-look-fast fails to handle alternative CUIs > > Thanks for the details Sean. I had assumed the conversion to Long was related > to sort/search efficiency but that makes sense.

RE: dictionary-look-fast fails to handle alternative CUIs

2015-07-08 Thread Finan, Sean
4:21 PM To: dev@ctakes.apache.org Subject: Re: dictionary-look-fast fails to handle alternative CUIs Thanks for the details Sean. I had assumed the conversion to Long was related to sort/search efficiency but that makes sense. I had been thinking of something similar with parsing out the non

Re: dictionary-look-fast fails to handle alternative CUIs

2015-07-08 Thread britt fitch
; tables as longs takes up a lot less space than storing them as 8 character > strings. > > From: britt fitch [mailto:britt.fi...@wiredinformatics.com] > Sent: Wednesday, July 08, 2015 2:23 PM > To: dev@ctakes.apache.org > Subject: dictionary-look-fast fails to handle alternative

RE: dictionary-look-fast fails to handle alternative CUIs

2015-07-08 Thread Finan, Sean
] Sent: Wednesday, July 08, 2015 2:23 PM To: dev@ctakes.apache.org Subject: dictionary-look-fast fails to handle alternative CUIs This is largely directed to Sean but open to other feedback as well. The current fast lookup using a BSV parses the first field as “C” and up to 7 numerals, padding with

RE: dictionary-look-fast fails to handle alternative CUIs

2015-07-08 Thread Finan, Sean
5 2:23 PM To: dev@ctakes.apache.org Subject: dictionary-look-fast fails to handle alternative CUIs This is largely directed to Sean but open to other feedback as well. The current fast lookup using a BSV parses the first field as “C” and up to 7 numerals, padding with “0" as needed to re

dictionary-look-fast fails to handle alternative CUIs

2015-07-08 Thread britt fitch
This is largely directed to Sean but open to other feedback as well. The current fast lookup using a BSV parses the first field as “C” and up to 7 numerals, padding with “0" as needed to reach that length when applicable [see CuiCodeUtil.getCuiCode(String)] The CUI string is then substring’d fr