-
Notifications
You must be signed in to change notification settings - Fork 26
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
Energy Decomposition Analysis adopted for general MCSCF #64
Conversation
LGTM. Do you know if any of the tests actually test this feature? Perhaps we could add one if not. |
We don't have any test for this functionality. I will add an unit test. Here, the sum of EDA components would be equal to the MCPDFT energy for the given state. |
Not necessarily, I believe. Doesn't it also include more than just the Coulomb interaction, such as CAS exchange? As in |
Sorry, there is a test for energy_decomposition. Though, whatever I have changed is passing the test. Also, energy_decomposition is not implemented for the hybrid functionals. However, it won't be that difficult to implement it. It can be represented as |
Yes, you are right. MCPDFT energy doesn't include the e_ncwfn. |
You don't have to even do that. The MC-PDFT energy should actually be |
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 cleaning up my old spaghetti!
Previously EDA is hardcoded for the CASSCF wavefunction only. I added the functionality to do this analysis for the DMRG-PDFT as well.