-
Notifications
You must be signed in to change notification settings - Fork 10
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
Allow to pass keyword arguments to itensor mps estimator for fast computation. #203
Allow to pass keyword arguments to itensor mps estimator for fast computation. #203
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
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.
Thank you!
Please check my comments. I think the updates here can also be applied to Sampler. Can you do the same for it?
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
Sure, leave it to me! |
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
…ensor_mps_sampler`
Of course we can pass `mindim` as long as ITensors.apply supports it internally.
…that create a concurrent estimator
…ncurrent_sampler`
Done. Ready for review 🙏 |
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.
Thank you!
I have made a few minor comments. Please take care of it 🙏
add `.` Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
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.
Almost!
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
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.
Sorry the documentation is still broken.
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
Co-authored-by: Toru Shibamiya <49753430+toru4838@users.noreply.github.com>
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.
Great work!
Description
This PR allows users to pass keyword arguments to
jl.apply
in the auxiliary function _estimate. Note thatjl.apply
Julia function ITensors.apply. By passingmaxdim
, it enables us to calculate the expectation value at high speed by obtaining the Truncated SVD in ITensors.factorize and svd. These functions are called from theITensors.apply
.