-
Notifications
You must be signed in to change notification settings - Fork 3
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
working ccd imaginary time propagation #30
base: master
Are you sure you want to change the base?
Conversation
might need to change the indices of the C-matrix?
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.
If you remove the unused import of the RungeKutta4
integrator, black the files and import ITDCCD
in coupled_cluster/__init__.py
, we'll merge :)
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.
Nice! Do a final merge with master and black, and we're there.
This has been broken by the refactoring of our time-dependent solvers. However, with the new structure, a very simple implementation of imaginary time propagation would be to have a property of the time-dependent solver, I'll close this PR and open a new one (which will contain considerably less code) if ya'll agree:) |
That sounds like a good idea! |
Straight from the hat.