Increase default threshold for all cycle detection.

johnmay · johnmay · commit 04f4d1647932 · 2021-01-17T21:10:02.000Z
diff --git a/base/core/src/main/java/org/openscience/cdk/graph/Cycles.java b/base/core/src/main/java/org/openscience/cdk/graph/Cycles.java
@@ -746,7 +746,7 @@ int[][] apply(int[][] graph, int length) {
             /** {@inheritDoc} */
             @Override
             int[][] apply(int[][] graph, int length) throws Intractable {
-                final int threshold = 684; // see. AllRingsFinder.Threshold.Pubchem_99
+                final int threshold = 3072; // see. AllRingsFinder.Threshold.Pubchem_994
                 AllCycles ac = new AllCycles(graph, Math.min(length, graph.length), threshold);
                 if (!ac.completed())
                     throw new Intractable("A large number of cycles were being generated and the"
@@ -807,7 +807,7 @@ int[][] apply(int[][] graph, int length) throws Intractable {
             /** {@inheritDoc} */
             @Override
             int[][] apply(int[][] graph, int length) throws Intractable {
-                final int threshold = 684; // see. AllRingsFinder.Threshold.Pubchem_99
+                final int threshold = 3072; // see. AllRingsFinder.Threshold.Pubchem_994
                 AllCycles ac = new AllCycles(graph, Math.min(length, graph.length), threshold);
 
                 return ac.completed() ? ac.paths() : VERTEX_SHORT.apply(graph, length);
@@ -978,8 +978,8 @@ private static final class AllUpToLength implements CycleFinder {
 
         private final int predefinedLength;
 
-        // see. AllRingsFinder.Threshold.Pubchem_99
-        private final int threshold = 684;
+        // see. AllRingsFinder.Threshold.Pubchem_994
+        private final int threshold = 3072;
 
         private AllUpToLength(int length) {
             this.predefinedLength = length;
