-
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
Status of MultiDiracDeterminantWithBackflow #826
Comments
It seems out-of-date and all the routines have a linked copy in MultiDiracDeterminantBase.cpp and MultiDiracDeterminantBase.2.cpp |
Note: We are recently using the backflow functionality, but with single determinants. Use of multideterminants with backflow is a proven improvement. Does MultiSlaterDeterminantWithBackflow work? |
Will remove. |
This was an initial (incomplete) attempt to implement the Woodbury formula for fast updates on top of backflow. Many years ago, I worked out how to do it for the gradient but not for the laplacian, so I never finished it.
Miguel
…Sent from my iPhone
On May 2, 2018, at 8:14 AM, jtkrogel ***@***.***> wrote:
MultiDiracDeterminantWithBackflow.cpp was identified as a dangling (unbuilt) file during automated searches for dead code. What is the status of this code/should it be kept? Is there any functionality here that is not covered in MultiSlaterDeterminantWithBackflow?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
As far as I know, now we have the following classes. MultiSlaterDeterminantFast is the main one used to do MSD. Probably MultiSlaterDeterminant can operate but slower. MultiSlaterDeterminantWithBackflow carries all the character of MultiSlaterDeterminant. To move forward, I think we need to add Backflow on top of the fast algorithm and remove the dummy ones. On the other hand, backflow can be largely improved by itself, I saw similar problems like the AoS Jastrows in the backflow. So I think first, we improve the backflow on single det and then bring it to the fast multidet class. |
It might be useful to have a slow (traditional) multiple determinant implementation available too. This is well defined topic that will need investigating as we learn about the effectiveness (cost, accuracy) of different ways to improve the nodal surfaces. |
Tests are requested by #844 |
MultiDiracDeterminantWithBackflow.cpp was identified as a dangling (unbuilt) file during automated searches for dead code. What is the status of this code/should it be kept? Is there any functionality here that is not covered in MultiSlaterDeterminantWithBackflow?
The text was updated successfully, but these errors were encountered: