No issues so far. I think you are already handling the 1 edge case I could come up with which was if the numeral portion of the code started with a 0 and it 0 was lost during the divide step but it looks like you are inserting leading zeros to the numeral portion if needed with digitCount.
I’ll definitely report back if I notice any performance change given the new logic though. Britt Fitch Wired Informatics 265 Franklin St Ste 1702 Boston, MA 02110 http://wiredinformatics.com britt.fi...@wiredinformatics.com > On Jul 10, 2015, at 5:31 PM, Finan, Sean <sean.fi...@childrens.harvard.edu> > wrote: > > 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 > > Thanks, just finished testing and closed the ticket. > > > > > > > > > > > Britt Fitch > Wired Informatics > 265 Franklin St Ste 1702 > Boston, MA 02110 > http://wiredinformatics.com > britt.fi...@wiredinformatics.com<mailto:britt.fi...@wiredinformatics.com> > > On Jul 9, 2015, at 3:44 PM, Finan, Sean > <sean.fi...@childrens.harvard.edu<mailto:sean.fi...@childrens.harvard.edu>> > wrote: > > 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<mailto:dev@ctakes.apache.org> > Subject: Re: dictionary-look-fast fails to handle alternative CUIs > > 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<mailto:britt.fi...@wiredinformatics.com<mailto:britt.fi...@wiredinformatics.com%3cmailto:britt.fi...@wiredinformatics.com>> > > On Jul 9, 2015, at 3:19 PM, britt fitch > <britt.fi...@wiredinformatics.com<mailto:britt.fi...@wiredinformatics.com<mailto:britt.fi...@wiredinformatics.com%3cmailto:britt.fi...@wiredinformatics.com>>> > wrote: > > Absolutely. I’ll create it now. >
signature.asc
Description: Message signed with OpenPGP using GPGMail