-
Notifications
You must be signed in to change notification settings - Fork 13
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
Improved Gauging Interface, TNO construction, Bug Fixes, Apply Function for a Vidal ITN #88
Conversation
Codecov Report
❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more. @@ Coverage Diff @@
## main #88 +/- ##
==========================================
+ Coverage 78.04% 79.25% +1.20%
==========================================
Files 62 61 -1
Lines 3289 3442 +153
==========================================
+ Hits 2567 2728 +161
+ Misses 722 714 -8
|
…check in combine_linkinds
Okay I incorporated the changes you suggested above! Do you know what is going on with the checks? Suddenly, for some reason certain packages aren't compiling? Even though I don't think I have changed the manifest/ dependencies. |
Looks like SciML/LinearSolve.jl#330. |
I see! That makes sense. |
We don't have any control over that issue, so as long as things pass in Julia 1.9 I think it's fine to merge this. |
Looks good, thanks! This will be very useful to have, I'm sure we'll find many use cases for it. |
Great to see this, thanks Joey and Matt!
On Fri, Jun 16, 2023 at 2:37 PM Matt Fishman ***@***.***> wrote:
Merged #88 <#88> into
main.
—
Reply to this email directly, view it on GitHub
<#88 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHJ3RLDVRKUFXDWVMUW5VTXLSRVNANCNFSM6AAAAAAYCM2D2M>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***
com>
--
-=Miles Stoudenmire=-
***@***.***
***@***.***
http://itensor.org/miles/
|
This PR adds a number of features in ITensorNetworks.
gauging.jl
). Functions are now available for belief propagation gauging a tensornetwork, starting in either symmetric or Vidal form. Functions are available for switching between these two forms, and for measuring the degree of `canonicalness' (i.e how well the itn satisfied certain canonical conditions) of a given itensornetwork (either in Vidal or Symmetric form).apply(o::ITensor, psi::ITensornetwork, bond_tensors::Datagraph)
function has been added toapply.jl
which performs a simple update on a tensornetwork in the Vidal gauge. Ideally this will eventually share more code withapply(o::ITensor, psi::ITensornetwork)
but for now I have left them more separate.tensornetworkoperators.jl
) for grouping up theITensors
in avector{ITensor}
object into groups of commutingITensors
(i.e. ones that act on different physical degrees of freedom). This is the workhorse of aget_tnos
function which builds a vector oftensornetworkoperator
s from a vector of ITensors.beliefpropagation.jl
to improve efficiency and avoid excessive calls to forming the union of two itensornetworks (which can be quite expensive).apply
function