-
Notifications
You must be signed in to change notification settings - Fork 21
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
Full and approximate polaritonic ADC energies up to second order #153
Open
BauerMarco
wants to merge
69
commits into
adc-connect:master
Choose a base branch
from
BauerMarco:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 61 commits
Commits
Show all changes
69 commits
Select commit
Hold shift + click to select a range
f886af7
qed adc1
BauerMarco b94d13f
qed-adc2 equations without adjustments to vector- and matrix class
BauerMarco 28235eb
added the lower order terms to all second order blocks
BauerMarco c64a6d6
full qed-adc2 converges for few states of all tested molecules and ba…
BauerMarco c906dc9
adc1 expanded to also include doubly excited photonic contributions
BauerMarco adc8695
still trying to fix the symmetry problem, which is probably not only …
BauerMarco 3495f9b
still trying to fix the symmetry problem
BauerMarco bdee15d
symmetry seems to work fine with restricted triples or singles
BauerMarco 744c2d6
Lanczos works for qed, applied massive improvement to guesses. Also D…
BauerMarco 7fdb127
check
BauerMarco ad0b144
check
BauerMarco 9bdca70
check
BauerMarco c2c9a1a
enabled qed-adc1 from qed-hf reference (for single + double photon di…
BauerMarco bf46dde
started correcting for wrong factors (cancellations should be done by…
BauerMarco b3580ac
some factors are still wrong
BauerMarco 380a582
QED-ADC(2) for QED-HF reference should work now
BauerMarco 0c56424
included check for unrestricted or restricted reference
BauerMarco 77a96ea
cashed some stuff in matrix.py
BauerMarco 8179a2b
corrected QED-ADC(2) from QED-HF reference (should be correct now and…
BauerMarco 2d028d5
removed .gs from the implementation (since it is always zero) and imp…
BauerMarco 001e2d3
added the possibility to only take first order photonic coupling into…
BauerMarco 7b8ac29
first_order_coupling option now also treats additive part to ERIs in …
BauerMarco 1866ce1
first_order_coupling option now also treats additive part to ERIs in …
BauerMarco 479ab98
uncommented missing H_1 terms in photonic coupling blocks
BauerMarco f1c9203
corrected mistake within the qed pphh_ph (and vice versa) blocks
BauerMarco ad3f8c7
corrected sign error in ph_pphh photonic coupling blocks
BauerMarco c60a4b0
full_diagonalize option included for qed-adc(1)
BauerMarco 0940196
added omega with corresponding factor to diagonal doubles space
BauerMarco b209615
added 1. order coupling blocks to first order coupling
BauerMarco 06c18fe
corrected the scaling factor for omega in the doubles block
BauerMarco 3a1d8b4
documentary stuff
BauerMarco d5e8867
changed factor and sign of H_1 term of photonic coupling blocks
BauerMarco 7c2fe87
again factor changes in H_1 part of photonic coupling blocks
BauerMarco 72a4fa8
see above
BauerMarco 8fc907f
QED-ADC(2) test works now
BauerMarco 911fb88
less printout in davidson
BauerMarco 6167f34
import for orbital energies and restricted checks in psi4 now works v…
BauerMarco 696269b
orbital energies in psi4 backend need to be adjusted, depending on in…
BauerMarco 41c8bab
adjusted psi4 backend for hilbert qed-(u/r)hf and psi4numpy cqed-rhf …
BauerMarco 5730053
merge check from qed_adc into master
40e752b
merged changes from master into submatrix class
99c78a9
AdcMatrix.py now has only one matrix class, which is capable of handl…
292d3f9
included approximation scheme, which seems to provide a tiny error, o…
9ff2131
minor changes
BauerMarco ee4be7e
searching for numerical inconsistency in approx function
BauerMarco 28ca773
numerical inconsistency in approx function found and removed
BauerMarco 89b12cc
reintroduced approx method
BauerMarco 8dfbb93
no printout from LazyMp anymore
BauerMarco aa44ebb
started with qed test
BauerMarco cb369d1
qed parameters can now also be set via the run_adc function. Tests fo…
BauerMarco 0d7d166
less printout and made the max subspace for qed a fully manual option…
BauerMarco c25cc1d
final cleanup for initial commit to master
BauerMarco f214171
Merge pull request #2 from BauerMarco/qed_merge_check
BauerMarco 671183b
removed mp2_diffdm for qed, since it has not been tested thoroughly yet
BauerMarco ec2f2dd
lgtm corrections
BauerMarco da9555a
making the linter happy
BauerMarco 9d5f74a
removed numerical inconsistency of approx method, by reintroducing th…
BauerMarco 1fabe94
adapted the test cases, so they also don't fail, if all tests are run…
BauerMarco 9cec484
apparently mp1 contribution got added twice in qed-mp2 energy from no…
BauerMarco e8a032d
qed approx method is now able to account for photon losses in the cav…
BauerMarco 45c7b8e
Merge pull request #3 from BauerMarco/qed_adc_complex_omega
BauerMarco af4b380
qed adc runs with AmplitudeVector object, instead of QED_AmplitudeVector
BauerMarco 529c6a6
further code cleanup, due to full qed handling via AmplitudeVector, a…
BauerMarco f9771b1
further cleanup
BauerMarco 21a1585
Merge branch 'adc-connect:master' into review_cycle1
BauerMarco 6edf332
Merge pull request #4 from BauerMarco/review_cycle1
BauerMarco 9b7b618
corrected qed_t1 denominator
BauerMarco 5bbb3eb
arbitrary spatial orientations for polaritonic frequency and coupling…
BauerMarco 889e213
make quasi-diabatic matrices accessible from resulting ExcitedStates …
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
merge this class with AmplitudeVector