Skip to content

Commit

Permalink
Revert "use Int32 instead UInt32 as index type in (sparse) projections"
Browse files Browse the repository at this point in the history
Indeed, it seems that we are actually hitting the typemax(Int32)
boundary in some cases and UInt32 is necessary here

This reverts commit ba9af48.
  • Loading branch information
kalmarek committed Jul 20, 2023
1 parent 2acb405 commit 8436d76
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions src/matrix_projections.jl
Original file line number Diff line number Diff line change
Expand Up @@ -174,8 +174,8 @@ function matrix_projection_irr_acc!(
weight,
)
iszero(weight) && return result
I = Int32[]
J = Int32[]
I = UInt32[]
J = UInt32[]
V = eltype(result)[]
for (val, cc) in zip(vals, ccls)
iszero(val) && continue
Expand Down Expand Up @@ -232,8 +232,8 @@ function matrix_projection_irr_acc!(
weight,
)
iszero(weight) && return result
I = Int32[]
J = Int32[]
I = UInt32[]
J = UInt32[]
V = eltype(result)[]
for (val, ccl) in zip(class_values, conjugacy_cls)
iszero(val) && continue
Expand Down Expand Up @@ -304,8 +304,8 @@ function matrix_representation_acc!(
},
)
b = basis(parent(α))
I = Int32[]
J = Int32[]
I = UInt32[]
J = UInt32[]
V = eltype(result)[]
for (idx, val) in StarAlgebras._nzpairs(StarAlgebras.coeffs(α))
g = b[idx]
Expand All @@ -326,8 +326,8 @@ function matrix_representation_acc!(
α::AlgebraElement,
)
b = basis(parent(α))
I = Int32[]
J = Int32[]
I = UInt32[]
J = UInt32[]
V = eltype(result)[]
for (idx, val) in StarAlgebras._nzpairs(StarAlgebras.coeffs(α))
iszero(val) && continue
Expand Down

0 comments on commit 8436d76

Please sign in to comment.