Skip to content

Commit a3be740

Browse files
committed
do not use BECKE_CUTOFF; fixes #18
this cutoff fails in particular for diffuse basis sets we need an adjustable cutoff
1 parent 5cba81c commit a3be740

File tree

3 files changed

+3943
-3942
lines changed

3 files changed

+3943
-3942
lines changed

include/parameters.h

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
#define PARAMETERS_H_INCLUDED
77

88
const double SMALL = 1.0e-14;
9-
const double BECKE_CUTOFF = 8.0;
9+
// const double BECKE_CUTOFF = 8.0;
1010
const int BECKE_HARDNESS = 3;
1111
const int MAX_ANGULAR_ORDER = 33;
1212
const int MAX_ANGULAR_GRID = 5810;

src/becke_partitioning.cpp

+6-5
Original file line numberDiff line numberDiff line change
@@ -146,11 +146,12 @@ double get_becke_w(const double center_coordinates[],
146146
dist_a = vx * vx + vy * vy + vz * vz;
147147
dist_a = std::sqrt(dist_a);
148148

149-
if (a != icent && dist_a > BECKE_CUTOFF)
150-
{
151-
pa[a] = 0.0;
152-
continue;
153-
}
149+
// in principle good idea but fails for larger molecules containing diffuse sets
150+
// if (a != icent && dist_a > BECKE_CUTOFF)
151+
// {
152+
// pa[a] = 0.0;
153+
// continue;
154+
// }
154155

155156
R_a = get_bragg_angstrom(center_charge[a]);
156157

0 commit comments

Comments
 (0)