diff options
| author | Thomas White <taw@physics.org> | 2020-10-26 15:06:38 +0100 |
|---|---|---|
| committer | Thomas White <taw@physics.org> | 2020-10-26 15:08:50 +0100 |
| commit | 2089e6e3c0a34430239dce596bc27db30611ac2f (patch) | |
| tree | 0b269fba30065b28c51ca3456313d26347fc58e5 /src/indexamajig.c | |
| parent | 6819e268a8401fec0001812e7612909b3211fab1 (diff) | |
Label all uses of cell tolerances
I'm sick of fixing this same issue over and over again.
New rule: any code handling unit cell tolerances MUST be labelled with
details of units.
Diffstat (limited to 'src/indexamajig.c')
| -rw-r--r-- | src/indexamajig.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/src/indexamajig.c b/src/indexamajig.c index 8285f780..2f96ecac 100644 --- a/src/indexamajig.c +++ b/src/indexamajig.c @@ -382,6 +382,8 @@ static error_t parse_arg(int key, char *arg, struct argp_state *state) break; case 401 : + /* Values in 'tols' are in frac (not %) and rad + * Conversion happens a few lines below */ r = sscanf(arg, "%f,%f,%f,%f,%f,%f", &args->iargs.tols[0], &args->iargs.tols[1], &args->iargs.tols[2], &args->iargs.tols[3], &args->iargs.tols[4], &args->iargs.tols[5]); @@ -620,12 +622,12 @@ int main(int argc, char *argv[]) args.iargs.cell = NULL; args.iargs.noisefilter = 0; args.iargs.median_filter = 0; - args.iargs.tols[0] = 0.05; - args.iargs.tols[1] = 0.05; - args.iargs.tols[2] = 0.05; - args.iargs.tols[3] = deg2rad(1.5); - args.iargs.tols[4] = deg2rad(1.5); - args.iargs.tols[5] = deg2rad(1.5); + args.iargs.tols[0] = 0.05; /* frac (not %) */ + args.iargs.tols[1] = 0.05; /* frac (not %) */ + args.iargs.tols[2] = 0.05; /* frac (not %) */ + args.iargs.tols[3] = deg2rad(1.5); /* radians */ + args.iargs.tols[4] = deg2rad(1.5); /* radians */ + args.iargs.tols[5] = deg2rad(1.5); /* radians */ args.iargs.threshold = 800.0; args.iargs.min_sq_gradient = 100000.0; args.iargs.min_snr = 5.0; |