support +/- AbstractFill from FillArrays #118
Conversation
Codecov Report
@@ Coverage Diff @@
## master #118 +/- ##
==========================================
+ Coverage 85.92% 85.96% +0.03%
==========================================
Files 6 6
Lines 732 734 +2
==========================================
+ Hits 629 631 +2
Misses 103 103
📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today! |
This reverts commit 34ce62b.
|
|
||
| @test convert(AbstractArray{Float64}, transpose(1:∞)) ≡ convert(AbstractArray{Float64}, transpose(oneto(∞))) ≡ | ||
| convert(AbstractMatrix{Float64}, transpose(1:∞)) ≡ convert(AbstractMatrix{Float64}, transpose(oneto(∞))) ≡ | ||
| AbstractMatrix{Float64}(transpose(1:∞)) ≡ AbstractMatrix{Float64}(transpose(oneto(∞))) ≡ | ||
| AbstractArray{Float64}(transpose(1:∞)) ≡ AbstractArray{Float64}(transpose(oneto(∞))) ≡ | ||
| transpose(InfUnitRange(1.0)) |
There was a problem hiding this comment.
Why doesn't this work anymore?
There was a problem hiding this comment.
I replaced InfUnitRange(1.0) with 1.0:∞ which is InfStepRange
There was a problem hiding this comment.
julia> typeof(float(1:5))
StepRangeLen{Float64, Base.TwicePrecision{Float64}, Base.TwicePrecision{Float64}, Int64}There was a problem hiding this comment.
Ah should InfUnitRange be restricted to integers?
There was a problem hiding this comment.
It's a subtype of OrdinalRange which by definition only accepts integers.
StepRange throws error for float input while UnitRange doesn't... I'm not sure
There was a problem hiding this comment.
It's a subtype of OrdinalRange which by definition only accepts integers.
I don't think there's a strict requirement on element types, only that the step should be an integer. Floating point UnitRanges are currently allowed
There was a problem hiding this comment.
Floating point
UnitRanges are currently allowed
They are, but the only way to construct them is the default constructor. The tests convert eltypes, which always result in StepRangeLen in Julia Base.
works after Generalise Zeros +/- FillArrays.jl#219
Converting InfUnitRange to float type now gives InfStepRange. InfStepRange is not an ordinal range #109