Add comment about copying LinearAlgebra.I

Author: lkdvos

src/factorizations/eig.jl

@@ -107,6 +107,7 @@ for f in [:eig_full!, :eigh_full!]
           D[bI] = bD
           V[bI] = bV
         else
+          # TODO: this should be `V[bI] = LinearAlgebra.I`
           copyto!(@view!(V[bI]), LinearAlgebra.I)
         end
       end

src/factorizations/lq.jl

@@ -108,6 +108,7 @@ function MatrixAlgebraKit.lq_compact!(
       L[bI] = bL
       Q[bI] = bQ
     else
+      # TODO: this should be `Q[bI] = LinearAlgebra.I`
       copyto!(@view!(Q[bI]), LinearAlgebra.I)
     end
   end
@@ -139,6 +140,7 @@ function MatrixAlgebraKit.lq_full!(
       L[bI] = bL
       Q[bI] = bQ
     else
+      # TODO: this should be `Q[bI] = LinearAlgebra.I`
       copyto!(@view!(Q[bI]), LinearAlgebra.I)
     end
   end

src/factorizations/qr.jl

@@ -110,6 +110,7 @@ function MatrixAlgebraKit.qr_compact!(
       Q[bI] = bQ
       R[bI] = bR
     else
+      # TODO: this should be `Q[bI] = LinearAlgebra.I`
       copyto!(@view!(Q[bI]), LinearAlgebra.I)
     end
   end
@@ -142,6 +143,7 @@ function MatrixAlgebraKit.qr_full!(
       Q[bI] = bQ
       R[bI] = bR
     else
+      # TODO: this should be `Q[bI] = LinearAlgebra.I`
       copyto!(@view!(Q[bI]), LinearAlgebra.I)
     end
   end

src/factorizations/svd.jl

@@ -168,6 +168,7 @@ function MatrixAlgebraKit.svd_compact!(
       S[bI] = bS
       Vᴴ[bI] = bVᴴ
     else
+      # TODO: this should be `U[bI] = LinearAlgebra.I` and `Vᴴ[bI] = LinearAlgebra.I`
       copyto!(@view!(U[bI]), LinearAlgebra.I)
       copyto!(@view!(Vᴴ[bI]), LinearAlgebra.I)
     end
@@ -204,15 +205,18 @@ function MatrixAlgebraKit.svd_full!(
       S[bI] = bS
       Vᴴ[bI] = bVᴴ
     else
+      # TODO: this should be `U[bI] = LinearAlgebra.I` and `Vᴴ[bI] = LinearAlgebra.I`
       copyto!(@view!(U[bI]), LinearAlgebra.I)
       copyto!(@view!(Vᴴ[bI]), LinearAlgebra.I)
     end
   end
 
   # Complete the unitaries for rectangular inputs
+  # TODO: this should be `U[Block(I, I)] = LinearAlgebra.I`
   for I in (blocksize(A, 2) + 1):blocksize(A, 1)
     copyto!(@view!(U[Block(I, I)]), LinearAlgebra.I)
   end
+  # TODO: this should be `Vᴴ[Block(I, I)] = LinearAlgebra.I`
   for I in (blocksize(A, 1) + 1):blocksize(A, 2)
     copyto!(@view!(Vᴴ[Block(I, I)]), LinearAlgebra.I)
   end