> 3) Handling of unpsecified stereochemistry in SMILES-SDF roundtrip test > > Using the SDF file format for the rountrip tests is also problematic. > Although we correctly write out unspecified atom stereo parities, these are > ignored (by default) when reading … > > Does this sound reasonable? If so, I'll implement this. This is currently > holding me back from being able to test on larger datasets.
Yes, I can imagine other cases where a 3D SD file exists but may have unspecified stereo - it should likely be the default. This is a useful feature for validation purposes (i.e., we generated a 3D geometry but it's an unspecified stereo center). > I currently have a patch for this which makes a copy of the molecule. This > could be made more efficient but in the short term, it would already be an > improvement. … > > In any case, finding a solution to 3 will help to robustly test a more > efficient version of this fix. Also, the CheckStereoContraints could be made > more efficient, but I'd like to focus on making it work correctly first. The main reason for implementing the distance geometry method was as a fallback for the fragment-based builder (i.e., where the stereo can't be corrected, a fragment doesn't exist, etc.) - it does not need to be fast since the fragment builder is fast. It does need to be correct, which has been the problem with the existing gen3d use. In short, your analysis seems solid and I'd happily review any patches to these issues for a 3.0.1 release. Thanks, -Geoff
_______________________________________________ OpenBabel-Devel mailing list OpenBabel-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openbabel-devel
