FCI and GENCI initial core guess determinants for Davidson-Liu solver#377
Open
marink2 wants to merge 16 commits intoevangelistalab:mainfrom
Open
FCI and GENCI initial core guess determinants for Davidson-Liu solver#377marink2 wants to merge 16 commits intoevangelistalab:mainfrom
marink2 wants to merge 16 commits intoevangelistalab:mainfrom
Conversation
specific state core guess for fci module
tests for core guess
added docs
huangm29
reviewed
Jan 30, 2024
| guess_dets.push_back(det); | ||
| } | ||
| } else { | ||
| guess_dets.push_back(det); |
Collaborator
There was a problem hiding this comment.
Let us change push_back to emplace_back here, though the speed does not matter at this place. I can barely find a place where emplace_back is worse than push_back
huangm29
reviewed
Jan 30, 2024
|
|
||
| Default value: 10 | ||
|
|
||
| **CORE_GUESS** |
Collaborator
There was a problem hiding this comment.
I think we need a better name for this. Core_guess does not make sense to me. Making it longer and clearer what it means.
Collaborator
|
Please ensure your "User Notes" and "Checklist" correspond to your changes. See other PR for examples. |
huangm29
reviewed
Jan 30, 2024
| } else { | ||
| repr += "_" + std::to_string(0); | ||
| } | ||
|
|
Collaborator
There was a problem hiding this comment.
repr += "" + std::to_string(core_guess ? 1 : 0);
huangm29
requested changes
Jan 30, 2024
Collaborator
huangm29
left a comment
There was a problem hiding this comment.
Make some changes suggested by comments and these are good to go!
huangm29
approved these changes
Jan 31, 2024
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
PR incorporates the feature to run a FCI (or GENCI) calculation(s) with core-excited determinants as the initial guess space for the Davidson-Liu solver. Core-excited determinants include determinant strings with single- and none-occupation on the first position.
User Notes
Added default boolean parameter (
bool core_guess) toStateInfoclass.Added protected boolean member variable (
core_guess_) toActiveSpaceMethodclass.ActiveSpaceSolverassigns values ofDL_CORE_INITIAL_GUESStoStateInfoobject,StateInfomaps this value toActiveSpaceMethod, which passes value to GENCI or FCI solver.Feature is implemented for
GENCIandFCIsolvers only.Included two test cases,
fci-core-1andfci-core-2, for the implemented feature.Added default boolean parameter (
bool core_guess) tofind_initial_guess_detmember function of Forte.This feature prints the value(s) of
DL_CORE_INITIAL_GUESSin an appropriate place ("Initial Guess") of the output.dat file. To avoid cluttering the parameter list of this function consider creating an instance ofActiveSpaceMethodinsparce_ci_initial_guess.cc.Checklist
output.reffile