-
Notifications
You must be signed in to change notification settings - Fork 142
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
New Functor for Short Ranged en Jastrow #1680
Conversation
…uracy nuclear cusps. no bug check yet though
Can one of the maintainers verify this patch? |
okay to test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need a unit test to be sure that the added functor won't break over time.
Please take src/QMCWaveFunctions/tests/test_user_jastrow.cpp as an example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EDIT: Had not seen Ye's comment before writing this. We agree in any case.
A unit test would be helpful.
@markdewing Did you ever try to build one for any of the other Functors? What would be the best piece of code for @eneuscamman to start from?
@eneuscamman Nice to see. Have you found this form particularly helpful so far or is this more experimental?
PRE.error("Unrecognized value for \"optimize\". Should be either yes or no", true); | ||
} | ||
|
||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add an abort here if the cname is unrecognized i.e. There is a typo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far it is experimental, but in early testing it does a better job than using a short-ranged, finely spaced spline form. As we learn more, the preferred functional form for these purposes may evolve...
test_wf.cpp contains some tests for a Pade functor. Some of the values for comparison were computed by hand. |
OK, I'll try to see to these things sometime this week... |
I've added the unit test and the extra error trap that Paul wanted. Let me know if there's anything else to do. |
@eneuscamman Can you remake opt.tex from the current version on develop or otherwise fix the conflict? opt.tex was edited and has modest changes throughout. You should be able to put in your new section again easily. |
@prckent The conflict in the manual optimization section is now resolved. |
@ye-luo The unit test is ready. I think this is ready to merge. |
Test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the updates.
I have implemented a new functor intended to act as an optimizable, short-ranged one-body Jastrow factor for correcting Gaussian orbitals in the cusp region. Unlike the pre-computed cusp correction, this functor can respond to changes in the wave function during VMC optimization, which we expect to be important as we begin to explore core excitations in first row atoms. With a core excitation, the electron density near the nucleus changes dramatically, and the resulting orbital relaxations should be accompanied by the ability to change the detailed behavior near the nucleus while maintaining exact cusp conditions. This functor seeks to provide this capability.