Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix converting empty tensors to array #180

Merged
merged 10 commits into from
Nov 21, 2024
Merged

Fix converting empty tensors to array #180

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
146d825
Fix conversion of empty tensor
lkdvos Nov 20, 2024
324da30
Add tests for empty tensor conversion
lkdvos Nov 20, 2024
6efd37b
Add `zero(::ElementarySpace)`
lkdvos Nov 20, 2024
53a2686
Fix failing tests
lkdvos Nov 20, 2024
389bcec
Add explicit test for issue #178
lkdvos Nov 20, 2024
88a7fec
Fix wrong type
lkdvos Nov 20, 2024
b143e5b
`eltype` should determine precision
lkdvos Nov 20, 2024
74dacc9
Add entry to docs
lkdvos Nov 20, 2024
6d93ae1
small fix
lkdvos Nov 20, 2024
d594dec
small fix attempt II
lkdvos Nov 21, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion src/tensors/abstracttensor.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -497,7 +497,7 @@ function Base.convert(::Type{Array}, t::AbstractTensorMap)
else
cod = codomain(t)
dom = domain(t)
T = isreal(sectorscalartype(I)) ? scalartype(t) : complex(scalartype(t))
T = sectorscalartype(I) <: Complex ? complex(scalartype(t)) : scalartype(t)
lkdvos marked this conversation as resolved.
Show resolved Hide resolved
A = zeros(T, dims(cod)..., dims(dom)...)
for (f₁, f₂) in fusiontrees(t)
F = convert(Array, (f₁, f₂))
Expand Down
Loading