Skip to content
This repository has been archived by the owner on May 4, 2019. It is now read-only.

address vcat return inconsistency #187

Closed
wants to merge 13 commits into from
Closed

address vcat return inconsistency #187

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
1689728
address https://github.com/JuliaStats/NullableArrays.jl/issues/167
cjprybol Feb 24, 2017
a36f6a4
reorder import statements
cjprybol Feb 24, 2017
5d9d982
move from operators files to nullablevector files
cjprybol Feb 24, 2017
4aa635c
reduce the piracy
cjprybol Feb 25, 2017
7d948a0
no more piracy!
cjprybol Feb 25, 2017
1b64de2
passing tests
cjprybol Feb 25, 2017
9cc8381
rearrange functions to cleanup comments
cjprybol Feb 25, 2017
b8c87b5
spacing
cjprybol Feb 25, 2017
bb0b1ee
add more tests and reorder tests to match function order
cjprybol Feb 25, 2017
b8211a9
no more stack overflows
cjprybol Feb 25, 2017
d28b7b8
Int64 -> Int
cjprybol Feb 25, 2017
6c30d14
implement 2 argument cases and comment out 3 argument tests
cjprybol Mar 1, 2017
2e2b98a
correct spacing issue
cjprybol Mar 2, 2017
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
25 changes: 9 additions & 16 deletions src/nullablevector.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -314,26 +314,19 @@ function Base.empty!(X::NullableVector)
return X
end

function vcat(A::AbstractVector, B::AbstractVector...)
any(b -> isa(b, NullableArray), B) ?
typed_vcat(promote_eltype(A, B...), NullableArray(A), B...) :
typed_vcat(promote_eltype(A, B...), A, B...)
function vcat(A::AbstractVector, B::NullableVector)
typed_vcat(promote_eltype(A, B), NullableArray(A), B)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please reduce the number of spaces used for indentation here by 50%

end

function vcat(A::AbstractMatrix, B::AbstractMatrix...)
any(b -> isa(b, NullableArray), B) ?
typed_vcat(promote_eltype(A, B...), NullableArray(A), B...) :
typed_vcat(promote_eltype(A, B...), A, B...)
function vcat(A::AbstractMatrix, B::NullableMatrix)
typed_vcat(promote_eltype(A, B), NullableArray(A), B)
end

function vcat(A::AbstractArray, B::AbstractArray...)
any(b -> isa(b, NullableArray), B) ?
typed_vcat(promote_eltype(A, B...), NullableArray(A), B...) :
typed_vcat(promote_eltype(A, B...), A, B...)
function vcat(A::AbstractArray, B::NullableArray)
typed_vcat(promote_eltype(A, B), NullableArray(A), B)
end

function hcat(A::AbstractVecOrMat, B::AbstractVecOrMat...)
any(b -> isa(b, NullableArray), B) ?
typed_hcat(promote_eltype(A, B...), NullableArray(A), B...) :
typed_hcat(promote_eltype(A, B...), A, B...)
NullableVecOrMat = Union{NullableVector, NullableMatrix}
function hcat(A::AbstractVecOrMat, B::NullableVecOrMat)
typed_hcat(promote_eltype(A, B), NullableArray(A), B)
end
28 changes: 16 additions & 12 deletions test/nullablevector.jl
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -228,21 +228,25 @@ module TestNullableVector
@test typeof(vcat(1:2, NullableArray(3:4))) == NullableArray{Int,1}
@test typeof(vcat(NullableArray([1 2]), [3 4])) == NullableArray{Int,2}
@test typeof(vcat([1 2], NullableArray([3 4]))) == NullableArray{Int,2}
@test typeof(vcat(NullableArray(1:2), 3:4, 5:6)) == NullableArray{Int,1}
@test typeof(vcat(1:2, NullableArray(3:4), 5:6)) == NullableArray{Int,1}
@test typeof(vcat(1:2, 3:4, NullableArray(5:6))) == NullableArray{Int,1}
@test typeof(vcat(NullableArray([1 2]), [3 4], [5 6])) == NullableArray{Int,2}
@test typeof(vcat([1 2], NullableArray([3 4]), [5 6])) == NullableArray{Int,2}
@test typeof(vcat([1 2], [3 4], NullableArray([5 6]))) == NullableArray{Int,2}

@test typeof(hcat(NullableArray(1:2), 3:4)) == NullableArray{Int,2}
@test typeof(hcat(1:2, NullableArray(3:4))) == NullableArray{Int,2}
@test typeof(hcat(NullableArray([1 2]), [3 4])) == NullableArray{Int,2}
@test typeof(hcat([1 2], NullableArray([3 4]))) == NullableArray{Int,2}
@test typeof(hcat(NullableArray(1:2), 3:4, 5:6)) == NullableArray{Int,2}
@test typeof(hcat(1:2, NullableArray(3:4), 5:6)) == NullableArray{Int,2}
@test typeof(hcat(1:2, 3:4, NullableArray(5:6))) == NullableArray{Int,2}
@test typeof(hcat(NullableArray([1 2]), [3 4], [5 6])) == NullableArray{Int,2}
@test typeof(hcat([1 2], NullableArray([3 4]), [5 6])) == NullableArray{Int,2}
@test typeof(hcat([1 2], [3 4], NullableArray([5 6]))) == NullableArray{Int,2}

# add these back if we can find a general solution to propogating array type
# see https://github.com/JuliaLang/julia/issues/2326
# @test typeof(vcat(NullableArray(1:2), 3:4, 5:6)) == NullableArray{Int,1}
# @test typeof(vcat(1:2, NullableArray(3:4), 5:6)) == NullableArray{Int,1}
# @test typeof(vcat(1:2, 3:4, NullableArray(5:6))) == NullableArray{Int,1}
# @test typeof(vcat(NullableArray([1 2]), [3 4], [5 6])) == NullableArray{Int,2}
# @test typeof(vcat([1 2], NullableArray([3 4]), [5 6])) == NullableArray{Int,2}
# @test typeof(vcat([1 2], [3 4], NullableArray([5 6]))) == NullableArray{Int,2}

# @test typeof(hcat(NullableArray(1:2), 3:4, 5:6)) == NullableArray{Int,2}
# @test typeof(hcat(1:2, NullableArray(3:4), 5:6)) == NullableArray{Int,2}
# @test typeof(hcat(1:2, 3:4, NullableArray(5:6))) == NullableArray{Int,2}
# @test typeof(hcat(NullableArray([1 2]), [3 4], [5 6])) == NullableArray{Int,2}
# @test typeof(hcat([1 2], NullableArray([3 4]), [5 6])) == NullableArray{Int,2}
# @test typeof(hcat([1 2], [3 4], NullableArray([5 6]))) == NullableArray{Int,2}
end