Format and update workflows (#138)

Author: mtfishman

.github/workflows/FormatPullRequest.yml

@@ -0,0 +1,14 @@
+name: "Format Pull Request"
+
+on:
+  schedule:
+    - cron: '0 0 * * *'
+  workflow_dispatch:
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  format-pull-request:
+    name: "Format Pull Request"
+    uses: "ITensor/ITensorActions/.github/workflows/FormatPullRequest.yml@main"

.pre-commit-config.yaml

@@ -1,6 +1,9 @@
+ci:
+  skip: [julia-formatter]
+
 repos:
 - repo: https://github.com/pre-commit/pre-commit-hooks
-  rev: v4.5.0
+  rev: v5.0.0
   hooks:
   - id: check-merge-conflict
   - id: check-toml
@@ -9,6 +12,6 @@ repos:
     exclude_types: [markdown]  # incompatible with Literate.jl
 
 - repo: "https://github.com/domluna/JuliaFormatter.jl"
-  rev: v1.0.62  
+  rev: v2.1.0
   hooks:
   - id: "julia-formatter"

Project.toml

@@ -1,7 +1,7 @@
 name = "ITensorMPS"
 uuid = "0d1a4710-d33b-49a5-8f18-73bdf49b47e2"
 authors = ["Matthew Fishman <[email protected]>", "Miles Stoudenmire <[email protected]>"]
-version = "0.3.16"
+version = "0.3.17"
 
 [deps]
 Adapt = "79e6a3ab-5dfb-504d-930d-738a2a938a0e"

src/abstractmps.jl

@@ -694,7 +694,7 @@ end
 # TODO: change kwarg from `set_limits` to `preserve_ortho`
 function map!(f::Function, M::AbstractMPS; set_limits::Bool=true)
   for i in eachindex(M)
-    M[i, set_limits=set_limits] = f(M[i])
+    M[i, set_limits = set_limits] = f(M[i])
   end
   return M
 end
@@ -1306,7 +1306,7 @@ of the orthogonality center to avoid numerical overflow in the case of diverging
 See also [`normalize!`](@ref), [`norm`](@ref), [`lognorm`](@ref).
 """
 function normalize(M::AbstractMPS; (lognorm!)=[])
-  return normalize!(deepcopy_ortho_center(M); (lognorm!)=lognorm!)
+  return normalize!(deepcopy_ortho_center(M); (lognorm!)=(lognorm!))
 end
 
 """
@@ -2001,7 +2001,7 @@ function swapbondsites(ψ::AbstractMPS, b::Integer; ortho="right", kwargs...)
   elseif rightlim(ψ) > b + 2
     ψ = orthogonalize(ψ, b + 1)
   end
-  ψ[b:(b + 1), orthocenter=orthocenter, perm=[2, 1], kwargs...] = ψ[b] * ψ[b + 1]
+  ψ[b:(b + 1), orthocenter = orthocenter, perm = [2, 1], kwargs...] = ψ[b] * ψ[b + 1]
   return ψ
 end
 

src/lib/Experimental/src/dmrg.jl

@@ -10,7 +10,7 @@ function dmrg(
   operator, init::MPS; updater=eigsolve_updater, (observer!)=default_observer(), kwargs...
 )
   info_ref! = Ref{Any}()
-  info_observer! = values_observer(; info=info_ref!)
+  info_observer! = values_observer(; info=(info_ref!))
   observer! = compose_observers(observer!, info_observer!)
   state = alternating_update(operator, init; updater, observer!, kwargs...)
   return info_ref![].eigval, state

src/solvers/alternating_update.jl

@@ -104,7 +104,7 @@ function alternating_update(
       flush(stdout)
     end
     isdone = checkdone(;
-      state, sweep, outputlevel, observer=observer!, sweep_observer=sweep_observer!
+      state, sweep, outputlevel, observer=(observer!), sweep_observer=(sweep_observer!)
     )
     isdone && break
   end

src/solvers/sweep_update.jl

@@ -302,7 +302,7 @@ function region_update!(
     end
     set_nsite!(reduced_operator, nsite - 1)
     position!(reduced_operator, state, b1)
-    internal_kwargs = (; current_time, time_step=-time_step, outputlevel)
+    internal_kwargs = (; current_time, time_step=(-time_step), outputlevel)
     bond_reduced_state, info = updater(
       reduced_operator, bond_reduced_state; internal_kwargs, updater_kwargs...
     )
@@ -437,7 +437,7 @@ function region_update!(
     bond_reduced_state = state[b1]
     set_nsite!(reduced_operator, nsite - 1)
     position!(reduced_operator, state, b1)
-    internal_kwargs = (; current_time, time_step=-time_step, outputlevel)
+    internal_kwargs = (; current_time, time_step=(-time_step), outputlevel)
     bond_reduced_state, info = updater(
       reduced_operator, bond_reduced_state; internal_kwargs, updater_kwargs...
     )

src/solvers/tdvp.jl

@@ -79,7 +79,7 @@ function tdvp(
   nsweeps=nothing,
   nsteps=nsweeps,
   (step_observer!)=default_sweep_observer(),
-  (sweep_observer!)=step_observer!,
+  (sweep_observer!)=(step_observer!),
   kwargs...,
 )
   time_step, nsteps = time_step_and_nsteps(t, time_step, nsteps)

test/base/test_mpo.jl

@@ -562,7 +562,7 @@ end
     ψ = orthogonalize(ψ0, 2)
     A = prod(ITensors.data(ψ)[2:(N - 1)])
     randn!(A)
-    ψ[2:(N - 1), orthocenter=3] = A
+    ψ[2:(N - 1), orthocenter = 3] = A
     @test prod(ψ) ≈ ψ[1] * A * ψ[N]
     @test maxlinkdim(ψ) == 4
     @test ITensorMPS.orthocenter(ψ) == 3
@@ -728,7 +728,7 @@ end
     s = siteinds(H2; plev=1)
     C = combiner.(s; tags="X")
     H2 .*= C
-    H2H2 = prime(H2; tags=!ts"X") * dag(H2)
+    H2H2 = prime(H2; tags=(!ts"X")) * dag(H2)
     @test @disable_warn_order prod(HH) ≈ prod(H2H2)
   end
 

test/base/test_mps.jl

@@ -1334,7 +1334,7 @@ end
     ψ = orthogonalize(ψ0, 2)
     A = prod(ITensorMPS.data(ψ)[2:(N - 1)])
     randn!(A)
-    ψ[2:(N - 1), orthocenter=3] = A
+    ψ[2:(N - 1), orthocenter = 3] = A
     @test prod(ψ) ≈ ψ[1] * A * ψ[N]
     @test maxlinkdim(ψ) == 4
     @test ITensorMPS.orthocenter(ψ) == 3