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

Add callback solver example using dyson for EWDMET #164

Merged
merged 7 commits into from
Apr 12, 2024
Merged

Conversation

basilib
Copy link
Contributor

@basilib basilib commented Apr 11, 2024

Fixes issue when providing only the DMs from a callback solver in an SCMF calculation.
Adds example showing how to use dyson to return spectral moments from a callback solver.

Copy link

codecov bot commented Apr 11, 2024

Codecov Report

Attention: Patch coverage is 0% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 0.00%. Comparing base (e42af2f) to head (78ae86f).

❗ Current head 78ae86f differs from pull request most recent head 40407e6. Consider uploading reports for the commit 40407e6 to get more accurate results

Files Patch % Lines
vayesta/ewf/ewf.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master    #164   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files         156     156           
  Lines       20795   20797    +2     
======================================
- Misses      20795   20797    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@basilib basilib requested a review from ghb24 April 11, 2024 19:10

# User defined FCI solver - takes pyscf mf as input and returns RDMs
# The mf argment contains the hamiltonain in the orthonormal cluster basis
# Pyscf or other solvers may be used to solve the cluster problem and may return RDMs, CIID amplitudes or CCSD amplitudes
Copy link
Contributor

Choose a reason for hiding this comment

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

CIID -> CISD
Also, can you add "or (in this case) the Greens function moments of the cluster", to make it clear what is going on in this example.

@@ -8,6 +8,8 @@
from vayesta.misc.molecules import ring

# User defined FCI solver - takes pyscf mf as input and returns RDMs
# The mf argment contains the hamiltonain in the orthonormal cluster basis
# Pyscf or other solvers may be used to solve the cluster problem and may return RDMs, CIID amplitudes or CCSD amplitudes
Copy link
Contributor

Choose a reason for hiding this comment

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

CIID -> CISD

@basilib basilib merged commit 622ae39 into master Apr 12, 2024
5 checks passed
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