Simplify indexing options

mosflm-cell-nolatt-multi-refine-bad-retry.... it's all getting a bit
too much.  This commit removes all the options from the indexing methods
except for the prior information flags (cell/nocell and latt/nolatt).
All the other options are now given at the top level with new
indexamajig options, e.g. --no-multi and --no-retry.  They are applied
to all the methods, with no option to (say) use retry for one method and
not for another.

This also makes the "peak alignment test" default to OFF, and
multi-lattice indexing ON.  I think that the prediction refinement does
a good enough job of checking the validity of indexing solutions,
allowing us to turn off the alignment check which filters out most
multi-lattice solutions.

1 files changed, 3 insertions, 13 deletions

diff --git a/libcrystfel/src/felix.c b/libcrystfel/src/felix.c
index 6daf4408..4c12778b 100644
--- a/libcrystfel/src/felix.c
+++ b/libcrystfel/src/felix.c
@@ -222,16 +222,7 @@ static int read_felix(struct felix_private *gp, struct image *image,
 
 	fclose(fh);
 
-	if ( gp->indm & INDEXING_CHECK_PEAKS ) {
-		if ( !peak_sanity_check(image, image->crystals,
-		                        image->n_crystals) )
-		{
-			free_all_crystals(image);
-			return 0;
-		}
-	}
-
-    return n_crystals;
+	return n_crystals;
 }
 
 
@@ -672,9 +663,8 @@ void *felix_prepare(IndexingMethod *indm, UnitCell *cell,
 	if ( gp == NULL ) return NULL;
 
 	/* Flags that Felix knows about */
-	*indm &= INDEXING_METHOD_MASK | INDEXING_CHECK_PEAKS
-	       | INDEXING_USE_LATTICE_TYPE | INDEXING_USE_CELL_PARAMETERS
-	       | INDEXING_CONTROL_FLAGS;
+	*indm &= INDEXING_METHOD_MASK
+	       | INDEXING_USE_LATTICE_TYPE | INDEXING_USE_CELL_PARAMETERS;
 
 	gp->cell = cell;
 	gp->indm = *indm;