Re: [Cdk-devel] Fignerprinter performance

[Rajarshi G. <rg...@in...>]

On Sep 12, 2008, at 12:31 AM, Egon Willighagen wrote:

> On Fri, Sep 12, 2008 at 4:33 AM, Rajarshi Guha <rg...@in...>  
> wrote:
>>

>> If I understand correctly, this code will actually not convert 'Cl'
>> to 'X'. So as a result the path string that will be hashed will
>> contain a 'Cl'. I assume this is wrong :)
>
> Oi... how long had this code be like this... that surely should say
> returnSymbol = "Y" etc...
>
> Please file a bug report...

I went ahead and fixed it in my latest commit

>> If stuff can be regenerated, I'd like to commit this since it's a big
>> jump in performance (see http://rguha.wordpress.com/2008/09/11/cdk-
>> performance-measurements/)
>
> Yes, I think this would be a good thing; please coordinate with  
> Stefan.

Stefan, I have committed the new Fingerprinter code - this will  
likely break builder3D since the template fp's need to be  
regenerated. Right now it's commited to 1.2.x. Egon, can you merge it  
into trunk?

-------------------------------------------------------------------
Rajarshi Guha  <rg...@in...>
GPG Fingerprint: D070 5427 CC5B 7938 929C  DD13 66A1 922C 51E7 9E84
-------------------------------------------------------------------
So the Zen master asked the hot-dog vendor,
"Can you make me one with everything?"
         - TauZero on Slashdot