Fixes attempt II

Author: lkdvos

src/transfermatrix/transfer.jl

@@ -68,21 +68,21 @@ end
 # the transfer operation of a density matrix with a utility leg in its codomain is ill defined - how should one braid the utility leg?
 # hence the checks - to make sure that this operation is uniquely defined
 function transfer_left(v::MPSTensor{S}, A::MPSTensor{S}, Ab::MPSTensor{S}) where {S}
-    check_ambiguous_braiding(space(v, 2))
+    check_unambiguous_braiding(space(v, 2))
     @plansor v[-1 -2; -3] := v[1 2; 4] * A[4 5; -3] * τ[2 3; 5 -2] * conj(Ab[1 3; -1])
 end
 function transfer_right(v::MPSTensor{S}, A::MPSTensor{S}, Ab::MPSTensor{S}) where {S}
-    check_ambiguous_braiding(space(v, 2))
+    check_unambiguous_braiding(space(v, 2))
     @plansor v[-1 -2; -3] := A[-1 2; 1] * τ[-2 4; 2 3] * conj(Ab[-3 4; 5]) * v[1 3; 5]
 end
 
 # the transfer operation with a utility leg in both the domain and codomain is also ill defined - only due to the codomain utility space
 function transfer_left(v::MPOTensor{S}, A::MPSTensor{S}, Ab::MPSTensor{S}) where {S}
-    check_ambiguous_braiding(space(v, 2))
+    check_unambiguous_braiding(space(v, 2))
     @plansor t[-1 -2; -3 -4] := v[1 2; -3 4] * A[4 5; -4] * τ[2 3; 5 -2] * conj(Ab[1 3; -1])
 end
 function transfer_right(v::MPOTensor{S}, A::MPSTensor{S}, Ab::MPSTensor{S}) where {S}
-    check_ambiguous_braiding(space(v, 2))
+    check_unambiguous_braiding(space(v, 2))
     @plansor t[-1 -2; -3 -4] := A[-1 2; 1] * τ[-2 4; 2 3] * conj(Ab[-4 4; 5]) * v[1 3; -3 5]
 end
 

src/utility/utility.jl

@@ -134,21 +134,21 @@ end
 
 # TODO: remove once TensorKit supports this
 _twist(t::AbstractTensorMap, i::Int) = _twist(BraidingStyle(sectortype(t)), t, i)
-_twist(::SymmetricBraiding, t::AbstractTensorMap, i::Int) = t
+_twist(::Bosonic, t::AbstractTensorMap, i::Int) = t
 function _twist(::NoBraiding, t::AbstractTensorMap, i::Int)
-    check_ambiguous_braiding(space(t, i))
+    check_unambiguous_braiding(space(t, i))
     return t
 end
 _twist(::BraidingStyle, t::AbstractTensorMap, i::Int) = twist(t, i)
 
 # Verify that the braiding is unambiguous: either through symmetric braiding or because the
 # sectors are all trivial
-function check_ambiguous_braiding(::Type{Bool}, V::VectorSpace)
+function check_unambiguous_braiding(::Type{Bool}, V::VectorSpace)
     I = sectortype(V)
     BraidingStyle(I) isa SymmetricBraiding && return true
     return all(isone, sectors(V))
 end
-function check_ambiguous_braiding(V::VectorSpace)
-    return check_ambiguous_braiding(Bool, V) ||
+function check_unambiguous_braiding(V::VectorSpace)
+    return check_unambiguous_braiding(Bool, V) ||
            throw(ArgumentError("cannot unambiguously braid $V"))
 end