apply code suggestions

Author: lkdvos

src/factorizations/factorizations.jl

@@ -58,12 +58,12 @@ LinearAlgebra.svdvals!(t::AbstractTensorMap) = diagview(svd_vals!(t))
 function MAK.ishermitian(t::AbstractTensorMap; kwargs...)
     return InnerProductStyle(t) === EuclideanInnerProduct() &&
         domain(t) == codomain(t) &&
-        all(cb -> MatrixAlgebraKit.ishermitian(cb[2]; kwargs...), blocks(t))
+        all(cb -> MAK.ishermitian(cb[2]; kwargs...), blocks(t))
 end
 function MAK.isantihermitian(t::AbstractTensorMap; kwargs...)
     return InnerProductStyle(t) === EuclideanInnerProduct() &&
         domain(t) == codomain(t) &&
-        all(cb -> MatrixAlgebraKit.isantihermitian(cb[2]; kwargs...), blocks(t))
+        all(cb -> MAK.isantihermitian(cb[2]; kwargs...), blocks(t))
 end
 LinearAlgebra.ishermitian(t::AbstractTensorMap) = MAK.ishermitian(t)
 
@@ -91,9 +91,5 @@ function MAK.is_right_isometric(t::AbstractTensorMap; kwargs...)
     f((c, b)) = MAK.is_right_isometric(b; kwargs...)
     return all(f, blocks(t))
 end
-function MAK.isisometric(t::AbstractTensorMap, (p₁, p₂)::Index2Tuple; kwargs...)
-    t = permute(t, (p₁, p₂); copy = false)
-    return MAK.isisometric(t; kwargs...)
-end
 
 end

test/autodiff/ad.jl

@@ -124,7 +124,6 @@ function remove_eighgauge_dependence!(
         ΔV, D, V; degeneracy_atol = MatrixAlgebraKit.default_pullback_degeneracy_atol(D)
     )
     gaugepart = project_antihermitian!(V' * ΔV)
-    gaugepart = (gaugepart - gaugepart') / 2
     for (c, b) in blocks(gaugepart)
         Dc = diagview(block(D, c))
         # for some reason this fails only on tests, and I cannot reproduce it in an
@@ -597,7 +596,7 @@ for V in spacelist
                     # Jacobian changes size if the truncation space changes, causing errors.
                     # So, first test the fixed space case, then do more limited testing on
                     # some gradients and compare to the fixed space case
-                    V_trunc = spacetype(t)(c => ceil(Int, min(size(b)...) / 2) for (c, b) in blocks(t))
+                    V_trunc = spacetype(t)(c => div(min(size(b)...), 2) for (c, b) in blocks(t))
                     trunc = truncspace(V_trunc)
                     USVᴴ_trunc = svd_trunc(t; trunc)
                     ΔUSVᴴ_trunc = (rand_tangent.(Base.front(USVᴴ_trunc))..., zero(last(USVᴴ_trunc)))