Skip to content
Merged
Show file tree
Hide file tree
Changes from 4 commits
Commits
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
8 changes: 8 additions & 0 deletions src/TensorStoreWrapper.jl
Original file line number Diff line number Diff line change
Expand Up @@ -150,6 +150,10 @@ function Base.setindex!(w::TensorStoreWrapper, v, indices...; kwargs...)
end

Base.size(w::TensorStoreWrapper) = pyconvert(Tuple, parent(w).shape)
function Base.size(w::TensorStoreWrapper, d::Integer)
d < 1 && throw(ArgumentError("dimension must be ≥ 1"))
return d <= ndims(w) ? size(w)[d] : 1
Comment thread
bjarthur marked this conversation as resolved.
Outdated
end
Base.ndims(w::TensorStoreWrapper) = pyconvert(Int, parent(w).rank)

const TS_TYPE_MAP = Dict(
Expand Down Expand Up @@ -228,6 +232,10 @@ end

# IndexDomainWrapper methods
Base.size(w::IndexDomainWrapper) = pyconvert(Tuple, parent(w).shape)
function Base.size(w::IndexDomainWrapper, d::Integer)
d < 1 && throw(ArgumentError("dimension must be ≥ 1"))
return d <= ndims(w) ? size(w)[d] : 1
Comment thread
bjarthur marked this conversation as resolved.
Outdated
end
Base.ndims(w::IndexDomainWrapper) = pyconvert(Int, parent(w).rank)
function Base.axes(w::IndexDomainWrapper)
rank = ndims(w)
Expand Down
8 changes: 8 additions & 0 deletions test/runtests.jl
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,10 @@ using PythonCall
@test eltype(w) == Int32
@test ndims(w) == 2
@test size(w) == (10, 20)
@test size(w, 1) == 10
@test size(w, 2) == 20
@test size(w, 3) == 1
@test_throws ArgumentError size(w, 0)
@test axes(w) == (1:10, 1:20)
end

Expand All @@ -44,6 +48,10 @@ using PythonCall
# Labeled indexing
sub_w = w[x=1:5, y=11:15]
@test size(sub_w) == (5, 5)
@test size(domain, 1) == 10
@test size(domain, 2) == 20
@test size(domain, 3) == 1
@test_throws ArgumentError size(domain, 0)
@test axes(sub_w) == (1:5, 11:15)

# translate_by
Expand Down
Loading