-
Notifications
You must be signed in to change notification settings - Fork 19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Value for H
radius does not agree with other codes
#77
Comments
I have no idea where the The H radius is not the only source of discrepancy between codes. For example, molgrid_defaults.cxx scales by
|
Ah, OK, so the half came from Becke's paper, not MHL. Good to know, its been awhile since I've thought about this. |
The Molpro definitions courtesy of @pjknowles are in https://gist.github.com/pjknowles/e25c5c12542da9a0fa25ad6cab4b459a Looks like they use a Bragg radius of 0.25 Å for H and He, which appears to be in agreement with Slater's value. That snippet of code also has tabulated values for the Mura-Knowles scheme, and the Treutler-Ahlrichs scheme, and Becke's modification of halving the Bragg radius discussed above. |
We take the value of the
H
radius (0.25 pm) from Slater, J, . J. Chem. Phys. 41, 3199, 1964.Pyscf
andNWChem
use 0.35 pm andChronusQ
uses the Bohr radius (0.529 pm, note factor of 2 in source). I have no strong opinion for what is "correct" here, but we need to expose the ability to make this flexible to ensure agreements between codes.N.B. This only affects MHL for the time being
Thanks @ajaypanyala, @dmejiar, @elambros, and @aodongliu for pointing this out
The text was updated successfully, but these errors were encountered: