-
Notifications
You must be signed in to change notification settings - Fork 141
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
Variance minimization failure between v3.6.0 and v3.7.0 #2496
Comments
Files needed to reproduce: |
The reported variance is increased! :-( Can you check the output of the first cycle of the optimizer? Does v3.7.0 believe to have reached a similar minimum to <=v3.6.0, or is a failure of plumbing already visible? |
It is worth noting that this uses a spline wavefunction and the quartic optimizer. There should be no excitement or unexpected results with this combination. |
The first series (series 0) is just measuring the inputted Jastrow. Here is what I get when I run VMC instead of OPT with the same input:
The energy variance reported to the log file appears to be wrong for 3.7.0 and later. The variance reported in the log is much larger than the actual variance. It is reported as decreasing, but obviously the internally computed variance is faulty:
|
Also confirming this is not an AoS/SoA issue as both have the same behavior. |
I don't know if anyone else was checking this, but I did a quick git bisect on this to track down the issue. It looks like the first bad commit was 16845f9, which was part of #1404. Attached are some files showing the data from the bisect |
@camelto2 thanks for corner the commit. I will have a look. |
Thanks Cody. |
hi all, any progress here? ver 392 C-HF-M1 series 0 -5.341298 ± 0.000776 0.876985 ± 0.003972 0.1642 FINAL VMC ver360 C-HF-M1 series 0 -5.399805 ± 0.000171 0.078521 ± 0.001085 0.0145 FINAL VMC |
We'll get on it. This has to be a simple plumbing failure. |
@dubeckym oneshiftonly is doing only energy minimization. So I need to take a close look. Could you attach a reproducer? |
Hopefully the same change accounts for the problems both with oneshift and variance minimization. Agreed that examples are needed for both. |
hi, thanks! this example shows the problem: |
@dubeckym Thank you. I will investigate both problems together. |
dear Ye Luo, |
Thanks for the quick fix Ye and thanks for letting us know it works @dubeckym . As of a few moments ago, the fix has been merged to develop. |
I noticed that QMCPACK currently struggles badly to optimize a Jastrow by variance minimization for a simple oxygen atom with a fairly short ranged two-body Jastrow (J1 rcut=3.5, J2 rcut=6.0). Even when handed a good Jastrow optimized via an older build, the optimizer still drifts away to extremely poor wavefunctions.
Below is a sequence of optimizations performed for QMCPACK versions spanning 3.5.0 to 3.9.2. The behavior of the code is reproducible before and after the breakage, which was introduced somewhere between v3.6.0 and v3.7.0:
The text was updated successfully, but these errors were encountered: