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
tests to check in. Would you like to write the jira > item? > > From: britt fitch [mailto:britt.fi...@wiredinformatics.com > <mailto:britt.fi...@wiredinformatics.com>] > Sent: Thursday, July 09, 2015 8:55 AM > To: dev@ctakes.apache.org <mailto:dev@ctakes.apache.org> >

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
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-numerals and converting them to 2-digit numeral values. i.e. “a” = 01, “z” = 26. Ultimately CN123456 wo

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

2015-07-08 Thread Finan, Sean
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 fitch [mailto:britt.fi...@wiredinformatics.com] Sen

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

2015-07-08 Thread Finan, Sean
Hi Britt, This did come up briefly wrt NCI custom cuis, but there was no urgency so it got shoved onto a back burner. I’m glad that you brought it up again as it is something that I’ve been wanting to enhance. My thought at the moment is to change CuiCodeUtil (…lookup2.util.) from a non-insta