Skip to content
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

Guess=Core broken (HeH+ test case) #2

Open
mbanck opened this issue Apr 21, 2014 · 0 comments
Open

Guess=Core broken (HeH+ test case) #2

mbanck opened this issue Apr 21, 2014 · 0 comments

Comments

@mbanck
Copy link

mbanck commented Apr 21, 2014

I've debugged the report by Noel O'Boyle in http://lists.nongnu.org/archive/html/freeon-users/2014-01/msg00000.html For me, only the HeH+ test case failed.

It appears NLink never gets initialized for the Guess=Core case in QCTC.F90, as ElectronsOn is set to .false. in this case and thus MakeRhoList() is never called. This leads to bogus NLink values like "11129" being passed to density::collate(), where the assert around line 211 (SUM(NZ(1:NExpt)).NE.NLink) fails.

If I set NLink to 0 in case MakeRhoList() is skipped in QCTC.F90 , the test case passes successfully (though I am not sure this is the correct fix). If I change the HeH+ test case to use Guess=Superpos as starting guess, it does not crash, either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant