Skip to content

Commit 86bb4cd

Browse files
committed
Fix ambiguity in Pivoted QR
1 parent 87337d8 commit 86bb4cd

File tree

3 files changed

+9
-2
lines changed

3 files changed

+9
-2
lines changed

Project.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
name = "RecursiveArrayTools"
22
uuid = "731186ca-8d62-57ce-b412-fbd966d074cd"
33
authors = ["Chris Rackauckas <[email protected]>"]
4-
version = "3.8.1"
4+
version = "3.8.2"
55

66
[deps]
77
Adapt = "79e6a3ab-5dfb-504d-930d-738a2a938a0e"

src/array_partition.jl

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -425,6 +425,13 @@ for factorization in vcat(__get_subtypes_in_module(LinearAlgebra, Factorization;
425425
end
426426
end
427427

428+
function LinearAlgebra.ldiv!(
429+
A::LinearAlgebra.QRPivoted{T, <: StridedMatrix{T}, <: AbstractVector{T}},
430+
b::ArrayPartition{T}) where {T <: Union{Float32, Float64, ComplexF64, ComplexF32}}
431+
x = ldiv!(A, Array(b))
432+
copyto!(b, x)
433+
end
434+
428435
function LinearAlgebra.ldiv!(A::LinearAlgebra.QRCompactWY{T, M, C},
429436
b::ArrayPartition) where {
430437
T <: Union{Float32, Float64, ComplexF64, ComplexF32},

test/linalg.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ for T in (UpperTriangular, UnitUpperTriangular, LowerTriangular, UnitLowerTriang
1818
@test B * Array(bbb) b
1919
end
2020

21-
for ff in (lu, svd, qr)
21+
for ff in (lu, svd, qr, Base.Fix2(qr, ColumnNorm()))
2222
FF = ff(A)
2323
@test A * (FF \ b) b
2424
bbb = copy(b)

0 commit comments

Comments
 (0)