From: John M. <joh...@gm...> - 2012-12-10 08:35:35
|
Hi Nina, I was thinking that too but I was apprehensive as there is already a configure and configureUnsetProperties. It might be more confusing to add another option? Perhaps that is the best way though. Thanks, J On 10 Dec 2012, at 06:34, Nina Jeliazkova <jel...@gm...> wrote: > > > On 9 December 2012 23:31, John May <jo...@eb...> wrote: > Hi All, > > I noticed that what I term auxiliary atom type properties, such as PI_BONDCOUNT, LONEPAIR_COUNT, SINGLE_COUNT… etc. are never configured. Some algorithms try to use this information (e.g. this line in CDKHueckelAromaticityDetector) but there is no obvious way to get at this information. It would be natural to add this to AtomTypeManipulator.configure() but as these are stored in a map it mean creating a HashMap on every single atom :/. > > It would be good to have these properties accessible but I can't see a useful way of doing it > > Perhaps a different flavour of AtomTypeManipulator.configure(), to set only these properties and to be used only when required? > > Regards, > Nina > > Any thoughts are most welcome. > > Many thanks, > J > John May | Predoctoral Student – Chemoinformatics and Metabolism > European Bioinformatics Institute, Wellcome Trust Genome Campus, Hinxton, Cambridge, CB10 1SD, UK > jo...@eb... | +44–(0) 1223 49 2603 > > > ------------------------------------------------------------------------------ > LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial > Remotely access PCs and mobile devices and provide instant support > Improve your efficiency, and focus on delivering more value-add services > Discover what IT Professionals Know. Rescue delivers > http://p.sf.net/sfu/logmein_12329d2d > _______________________________________________ > Cdk-devel mailing list > Cdk...@li... > https://lists.sourceforge.net/lists/listinfo/cdk-devel > > > ------------------------------------------------------------------------------ > LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial > Remotely access PCs and mobile devices and provide instant support > Improve your efficiency, and focus on delivering more value-add services > Discover what IT Professionals Know. Rescue delivers > http://p.sf.net/sfu/logmein_12329d2d_______________________________________________ > Cdk-devel mailing list > Cdk...@li... > https://lists.sourceforge.net/lists/listinfo/cdk-devel |