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

fix casscf-1 c1 #372

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix casscf-1 c1 #372

wants to merge 2 commits into from

Conversation

lcyyork
Copy link
Contributor

@lcyyork lcyyork commented Dec 14, 2023

Description

Add screening to remove small gradients (< 1.0e-12) for test case casscf-1.

User Notes

  • Features added
  • Changes to compilation (if any)

Checklist

  • Added/updated tests of new features and included a reference output.ref file
  • Removed comments in code and input files
  • Documented source code
  • Checked for redundant headers
  • Checked for consistency in the formatting of the output file
  • Documented new features in the manual
  • Ready to go!

@lcyyork lcyyork requested a review from fevangelista December 14, 2023 02:56
@fevangelista
Copy link
Member

I think this makes sense as a solution, but I would add that in general it is good to have a MCSCF that can break symmetry (if this is what is happening). I did not have time to look at this yet, but I would like to see what happens to the final orbitals when this patch is not applied. Let me put this on hold for a second.

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

Successfully merging this pull request may close these issues.

2 participants