-
Notifications
You must be signed in to change notification settings - Fork 25
Simple update for PEPO (3-site version) #262
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests.
🚀 New features to boost your workflow:
|
lkdvos
left a comment
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.
Overall looks good to me. I see you changed from AbstractTensorMap{<:Any,<:Any,1,N} to AbstractTensorMap{<:Any,<:Any,N,1}, is this just more convenient, or was there any special reason for that?
|
The truncation on the 3-site "cluster" is basically an OBC MPS truncation problem, so I decided to follow MPSKit convention in defining tensors in the cluster. (I was not aware of |

This PR adds the 3-site version of simple update for PEPO, which deals with Hamiltonians containing up to next-nearest neighbor terms. The J1-J2 model is used to test the algorithm by comparing the calculated finite-temperature energy (both tr(ρH) and ⟨ρ|H|ρ⟩) with high-temperature expansion results in https://doi.org/10.1103/PhysRevB.67.014416 (as @ogauthe did in #215).