ahwillia/catviews.jl

Concatenated Array views in Julia.

catviews.jl's Issues

`eachindex` of CatView is counterintuitive

If the idea is to have non-allocation vcat, then eachindex() should reutrn OneTo(length) other wise eachindex(collect(catview)) != eachindex(catview)

Tag a release

Can you tag a release?
I'ld like the fixes from 6 months ago

Unroll loops in getindex/setindex

I assumed the compiler would do this, but it doesn't.

The loops in getindex and setindex are small and fixed in size.
Since it is a generated function it is all set up to unroll them at compile time

Can we get this to work for CUDA?

Hello!

Would it be possible to do this example for a CuArray?

A = randn(10, 10); B = randn(10, 10); a = view(A, :); # no copying b = view(B, :); # no copying x = CatView(a, b); # no copying!!!

in CatView, in T, expected T<:Number, got Type{Vector{Float32}}

naively I'd think CatView can opt for simply using original eltype as new eltype?

reproducing:

julia> CatView([[1], [3], [4]], [[1,2,3],[4]])
ERROR: TypeError: in CatView, in T, expected T<:Number, got Type{Vector{Int64}}
Stacktrace:

Parameter Ordering not consistent with AbstractArray

To use CatView in my application, I need the following to be true:

CatView{Float64, 2} <: AbstractArray{Float64,2}

It looks like the parameter order is backwards compared to AbstractArray.

Would that be possible to fix?

Note that the following is true:
CatView <: AbstractArray

how to use CatView like hcat?

It seems Catview() behaves like vcat(). Is there a way to use it like hcat().

Failed precompilation

Hello! There seems to be a problem with precompilation.

julia> using CatViews
INFO: Recompiling stale cache file /home/ilya/.julia/lib/v0.6/CatViews.ji for module CatViews.

WARNING: deprecated syntax "typealias FastContiguousSubArray{T,N,P,I<:Tuple{Union{Colon,UnitRange},Vararg{Any}}} SubArray{T,N,P,I,true}" at /home/ilya/.julia/v0.6/CatViews/src/splitview.jl:30.
Use "FastContiguousSubArray{T,N,P,I<:Tuple{Union{Colon,UnitRange},Vararg{Any}}} = SubArray{T,N,P,I,true}" instead.
WARNING: The call to compilecache failed to create a usable precompiled cache file for module CatViews. Got:
WARNING: Module Iterators uuid did not match cache file.
ERROR: LoadError: Declaring __precompile__(true) is only allowed in module files being imported.
Stacktrace:
 [1] __precompile__(::Bool) at ./loading.jl:335
 [2] __precompile__() at ./loading.jl:331
 [3] include_from_node1(::String) at ./loading.jl:569
 [4] eval(::Module, ::Any) at ./boot.jl:235
 [5] _require(::Symbol) at ./loading.jl:483
 [6] require(::Symbol) at ./loading.jl:398
while loading /home/ilya/.julia/v0.6/CatViews/src/CatViews.jl, in expression starting on line 1

julia> versioninfo()
Julia Version 0.6.1-pre.0
Commit dcf39a1dda (2017-06-19 13:06 UTC)
Platform Info:
  OS: Linux (x86_64-linux-gnu)
  CPU: Intel(R) Core(TM) i7-4510U CPU @ 2.00GHz
  WORD_SIZE: 64
  BLAS: libopenblas (USE64BITINT DYNAMIC_ARCH NO_AFFINITY Haswell)
  LAPACK: libopenblas64_
  LIBM: libopenlibm
  LLVM: libLLVM-3.9.1 (ORCJIT, haswell)

julia> Pkg.status("Iterators")
 - Iterators                     0.3.1

Error tagging new release

The tag name "1.0.1" is not of the appropriate SemVer form (vX.Y.Z).
cc: @ahwillia