1.5.6 Release Notes

John May edited this page Apr 27, 2014 · 13 revisions

This release is the first using maven to build the project and lots of changes are related to reorganising directories and fixing build issues. New functionality includes an implementation of the popular extended connectivity fingerprints (ECFP) contributed by Alex Clark and Krishna Dole of [Collaborative Drug Discovery](Collaborative Drug Discovery). Additional changes include improved API utilities, descriptor fixes and improved stereochemistry perception / depiction. As always, a huge thank you to all contributors and reviewers.

Related blog posts

Download

Download options have slightly changed as external dependancies (libraries) are now resolved by maven.

Test status

This release brings a reorganisation of the project directory and build system. Some tests (ioformats) are disabled due to cyclic dependancies, maven also picked up a lot of tests that were not run and accounts for the increase in failures.

19961 (-760) tests (ioformats and coverage not run)
23 (+7) failures
0 (-19) coverage failures (disabled)
0 (-1) error

Authors

   112  John May
     7  Alex Clark
     5  Stephan Beisken
     4  Egon Willighagen
     3  Cyrus Harmon
     2  Krishna Dole
     1  Nina Jeliazkova

Reviewers

   1  Cyrus Harmon 
  71  Egon Willighagen 
  17  John May 

Full change log

  • Bumping version for release. 8443394
  • Added a missing space bd6e9e3
  • Site index page 523649e
  • Allow creation of staged site (without testapidocs) 160f671
  • Including Cyrus, Alex and Krishna as contributors. 50ca27a
  • Maven site reports developers as those with commit access. Other developers are listed as contributors. 39bddb6
  • Referenced in the doc/dict and now located there. f7b6051
  • Conversion table is related to atom types, no apparent usage. cc8d2fd
  • Git attribute files allowed the descriptor $Id$ to be tied to a commit. This was recently removed but may be useful again in future. 8dc5d0d
  • git ignored no longer needed ae2ea7c
  • Moving package documentation for ChemNomParser. a3d37b4
  • Element definitions are now an enum, this file has been moved to cdk-build-util. 483e649
  • HETATM type map for PDB reading. 46fbbbb
  • Unused CML test frameworks, referenced in io module. 9a40d0f
  • File not used, but now in correct module location. dd21e92
  • Residual config files left over from ant build. 9e3c5f5
  • Shade plugin requires maven v3.0+. 0bb1736
  • Update descriptor algorithms ontology. f0df4ea
  • Include simple javadoc tags. fcd9050
  • Consider implicit hydrogens in natural abundance utility. 9ee71fb
  • Consider implicit hydrogens in exact mass utility. 0e85729
  • Consider implicit hydrogen counts in getNaturalExactMass. 7433ac1
  • Fixed bug in CircularFingerprint/biotypes (for FCFP) 8f48540
  • Exceptions being thrown in functional fingerprint generation. 61648dc
  • Unpaired electrons affect hydrogen count. 36e2bcc
  • Triplet = 2 unpaired electrons, not currently represented. 658546d
  • Remove language assertion that caused test error. The test assertions were passing and the lang assert was causing an error randomly. This class will be moved to a deprecated module shortly. df644b0
  • Similarity measure was modified (d1527c6) but value was not updated. ebffa43
  • Check null hydrogen count [bug:1335] 536111b
  • Random test order causes failure now default option is to ignore aromaticity. 2f7e936
  • Pattern implementation for SMARTS queries. 1d51037
  • Move test resources to be local to the test that requires them. These files in particularly have been reused in pcore/io so the originals have been left in place for the moment. 55e30c6
  • We can remove the dependence on cdk-extra and valency check by not use the MoleculeFactory. Using cdk-data (test) isn't ideal but is better than extra. 17b5f5b
  • Dependencies that can be safely removed from cdk-group and cdk-hash. 898e3ae
  • Move the hash resources files to the same package as the test that needs them. 3e0b405
  • Move resources files required by cdh-hash to that module and eliminates dependence on testdata. 43e6ee3
  • We can correctly define the descriptors exported by each module. 73f9148
  • Utility in the Matching API provides simplified procedure to assigning a perfect matching. 454ed8f
  • Maximum matching using Edmond's blossom algorithm. 7f8e345
  • DisjointSetForest generally useful - move to cdk-standard. Left in the same package for now. 936ba2e
  • Storage and manipulation of an independent edge-set, a matching. A matching can be used to represent pi bond placement in conjugated/aromatic systems. a69010c
  • The Vertex Adjacency Magnitude descriptor requires the number of heavy atom - heavy atom bonds as input. The current incorrect calculation retrieves the number of heavy atoms instead. Correct bond number estimation and test cases have been added. f01df07
  • The rotatable bond count descriptor should not include C-N amides and terminal hydrogen- and hetero-atoms for the extended Lipinki's rule of five implementation (see Veber, D.F. et al., 2002. Journal of medicinal chemistry, 45(12), pp.2615–23). The parameters for the rotatable bond counts descriptor have been modified to that end and an option has been added to exclude simple C-N amides (ignoring tautomeric or charged constitutions). e7691f8
  • Container may be null (allowed) use the default length limit (|V|) from the graph instead. 07a41b2
  • A filter to only return cycles without a chord. 2d44f0a
  • Constructor should still be private - debugging from last commit. 590a4df
  • Explain the minimum length is chosen if a constructor and method limit are provided. 92538d1
  • Adapt the cycle finding to use the length. e9bdb3e
  • Parse a length limit to the cycle finding algorithm. f2adc55
  • Limit the size of initial cycles discovered. 7b4ef80
  • Limit the size of shortest paths discovered. 222a959
  • Allow five valent nitrogen for non-charge separated representation, see pyridinone test case. e2253d3
  • Check for normal valence in aromatic model before anything else. e7dafa6
  • Test that three valent nitrogen cations (radical) are not allow to be aromatic [bug:1332] 047b5e3
  • Ignore aromatic bonds by default. The option has be deprecated but should be removed in future. Global state, therefore impossible to test reliably. f66a59a
  • Correcting typos. 9fa7f10
  • Update README.md e7eb37c
  • Repository config to use RELEASE and LATEST. 3b16371
  • Empty targets are also a problem for the substructure search. Simple checks avoids the infinite loop. d9ef27f
  • Ignore this old test; this convention is not used anymore, AFAIK c1f7b2d
  • Providing a custom set of invariants before labelling. 8b7b722
  • Including JavaDoc for new methods. Would also be better to use Elements from a symbol, currently the method is case sensitive. 2fe701a
  • add new MolecularFormulaManipulator.getElementCount static methods that take (formula, IIsotope) and (formula, String) ee7907e
  • remove debug log statements in AtomPlacer.java ea62028
  • convert some AtomPlacer metohds to static methods and fixup their invocations adc857e
  • Descriptor $Id$ needs updating to new scheme. 14eaaf1
  • Use MDLV2000Reader for tests 8ef313f
  • Minor tidying of comments a11b89d
  • CircularFingerprinter recyclable 40c9854
  • Change implementation vendor to CDK, since that seems to be the convention. f5c0620
  • Minor edits to comments; remove unused debugging code and import. 3ff5d15
  • Updated ec35cce
  • Added missing files 282457b
  • Descriptors & fingerprints 125d674
  • Correcting repository id. 504105a
  • Correct version - previous release was 1.5.5 so the current version should be 1.5.6-SNAPSHOT. d4e81eb
  • Update README.md 026fe32
  • Section on the Maven artefacts and repository. 6f1bf62
  • Update README.md 5555882
  • Using markdown in README. a63405a
  • Configuring distribution management - these addresses are only needed for deployment to a remote repository. This repository may be different for each user and will also require a password. The locations can be defined in the ~/.m2/settings.xml file. 17956a2
  • Assembly of an uber-jar, dist-large. 65ca212
  • Correct unit test assertion. 1eb8ddd
  • Inheritance in tests causes problems for dependency analysis. We need to add cdk-diff and mockito to these modules even though the modules only have transitive interactions. 872b590
  • cdk-diff needed by the interfaces module. 256fd77
  • Using ‘mvm dependency:analyse’ on cdk-base modules to remove dependencies that were declared but not used and include dependencies that were used but not declared. In the second case these occur when the code interacts with a dependency of a dependency. Examples seen here are xml-apis (a dependency of xom) and hamcrest-core (a dependency of hamcrest-all). 2beecf0
  • Centralising hamcrest version in parent pom. 3e519b0
  • Centralising mockito version in parent pom. f5898b6
  • Centralising JUnit version in parent pom. ad9a726
  • Centralising Guava version in parent pom. e4827ae
  • Centralising Beam dependency version in the parent pom. a380ed3
  • Ignore config files from Eclipse, which it now autogenerates from Maven details 8195792
  • Default to the CDK version, if not other specificationIdentifier is given ac226a0
  • Update README 7b77e98
  • Update README f9ee3bf
  • Updating readme with maven instructions. 4836af6
  • Bumping copyright year. e4dd531
  • Added project names for each module (same as module name) 9ed3ce7
  • Moving testdata resources to the correct location. f4c3fa3
  • Using latest version of Beam. 4dcfe79
  • Dependencies are now resolved using Maven. Version is now in root pom.xml instead of build.props. 2d06a25
  • Ignore maven build directory (target/). 70f4e97
  • Removing old class from JChemPaint. 88f74e6
  • Module build files. 2b4ff75
  • @cdk.set files - note qsar set is now redundant and picked up from META-INF/services 08fbfc8
  • Redundant classes. 06096d3
  • Moving source files to separate source trees. 5922f5f
  • Moved to cdk-build-util project 09a046b
  • files to remove 1b65e6a
  • Aromaticity information is lost on descriptor execution unless the 'aromaticity parameter' is set to true. Already perceived -- pre-perceived -- aromaticity information should not be lost automatically if this parameter is set to false. Perception of unset parameters only via the helper method in the AtomContainerManipulator rectifies this issue. Three test cases: 1) Benzene with pre-perceived aromaticity and no aromaticity perception in the descriptor. 2) Benzene with no pre-perceived aromaticity and no aromaticity perception in the descriptor. 3) Benzene with no pre-perceived aromaticity and perception in the descriptor. 13c5dc2
  • Arbitrary labels in SMILES bracket atoms. The label is set in the pseudo atom. ed01edc
  • Beam 0.5. - resolve a bug that would randomly fail when configuring double-bond stereochemistry (found from the new random order of IStereoElement sets). - parse tetrahedral selenium and sulphur cations (okay according to inchi tech manual) - parser arbitrary labels in bracket atoms - they are never generated output but instead placed in the ‘label’ of a pseudo atom if one wants to handle them. 02c17f2
  • Avoid erroneous aromaticity due to abnormal valence. d4255b3
  • Atoms are not added to IQueryAtomContainer causing an index out of bounds exception downstream. Adding the atoms fixes the issue. 1a3f3d3
  • Removed redundant System.out statement. 0eefcdc
  • Simplified creation of a custom 'fall-back' cycle finder. 65a1a0a
  • Find all cycles up to (and including) a specified length. ea8f63b
  • Point to correct usage in error message 77abe62
  • Remove deprecation warning - this will likely be the best way to set the parameters for SMARTS for a while. f6e73fe
  • Ensure an exception is not thrown when a tetrahedral centre has an implicit hydrogen by the target does not. We want matching to be consistent whether the query as implicit/explicit hydrogens and so we say this is not a substructure match (with stereochemistry). ac470c1
  • Implicit hydrogen count should always be 0 or greater. 4ee04b7
  • Correct filtering of unique atom and bond matches. The filter now uses a new predicate for each successive iterations. 602a40e
  • Also initialise descriptors loaded by name. af67de8
  • Can’t run devel.xml but this should be working correctly. 700f50c
  • Open JavaDoc Checker now working with the doc tests in cdk-build-utils. We can also remove the compilation taglets / docchecks. d8333a4
  • Custom taglets all work correctly including the IOOptions - the ‘cheminf.bibx’ is now duplicated in the ‘cdk-build-util’ project. a7e9b99
  • Incuding the utils in the classpath allows project to build correctly. 62a633a
  • Remove net.sf.* classes - now in ‘cdk-build-utils’ 6e4bc3a
  • Check for null input before closing the resource. 94052ba
  • Don't stop the world if a single invalid molecule is found. The iterator is meant for parsing large data sets and should not stop early if an invalid structure was found. If the SMILES could not be parsed the iterator now returns an empty container and set the attempted input as a property. df9a307
  • Empty molecules are rare but found - this should not halt the reader. ade0307
  • input.ready() doesn't really do anything. 4756507
  • Deprecating redundant method. a528e81
  • Bumping version number - open for patching. ca1d1eb