Skip to content

Conversation

JonathonMisiewicz
Copy link
Contributor

Description

Replace an old-style call to moinfo_ in cc with something more modern: using Matrix technology to bypass the loop over the number of irreps.

Checklist

  • cc tests pass

Status

  • Ready for review
  • Ready for merge

Copy link
Contributor

@lothian lothian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you also zeroing out the diagonal for FMI in the UHF case? I looks like only the Fmi matrix is cleared there.

@JonathonMisiewicz
Copy link
Contributor Author

Are you also zeroing out the diagonal for FMI in the UHF case? I looks like only the Fmi matrix is cleared there.

Yes, on lines 75-77. Look for the little patch of green between the red.

@lothian
Copy link
Contributor

lothian commented Aug 25, 2022

Are you also zeroing out the diagonal for FMI in the UHF case? I looks like only the Fmi matrix is cleared there.

Yes, on lines 75-77. Look for the little patch of green between the red.

OK, now I see what you're doing. My only suggestion is that you modify the comment under RHF to read:

// This is because the amplitude update in RHF uses the full residual while the ROHF and UHF updates separate out the diagonal contributions from the Fock matrix [cf. Eqs. (1) and (2) of Stanton et al., J. Chem. Phys. 94, 4334-4345 (1991)].

@JonathonMisiewicz
Copy link
Contributor Author

Mostly adopted : I changed a few words to specify that these are true of Psi's implementations, not necessarily all implementations.

Thanks for the review! I will always accept specific journal references.

@hokru hokru merged commit 96b211d into psi4:master Aug 25, 2022
@JonathonMisiewicz JonathonMisiewicz deleted the moinfo branch August 25, 2022 14:28
@JonathonMisiewicz JonathonMisiewicz added cc For all issues involving the CC module, ground-state energies to response properties. cleanup For issues where the goal is to make Psi4 a little cleaner. labels Aug 25, 2022
@JonathonMisiewicz JonathonMisiewicz added this to the Psi4 1.7 milestone Aug 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cc For all issues involving the CC module, ground-state energies to response properties. cleanup For issues where the goal is to make Psi4 a little cleaner.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants