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.

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

Odd type bug (?) related to new call syntax #8851

Closed
timholy opened this issue Oct 30, 2014 · 3 comments
Closed

Odd type bug (?) related to new call syntax #8851

timholy opened this issue Oct 30, 2014 · 3 comments
Assignees
Labels
kind:bug Indicates an unexpected problem or unintended behavior

Comments

@timholy
Copy link
Sponsor Member

timholy commented Oct 30, 2014

I haven't followed this carefully enough to know whether this is something I need to change in my code, or whether it's a bug in julia, but my preliminary investigation suggests the latter. Problem was initially reported by PkgEvaluator, timholy/Grid.jl#51. The error message seems quite strange: in this line, y should be a Vector, not an InterpGrid{Float64,1,BCreflect,InterpNearest}. Moreover, those same lines work fine when run at the REPL:

julia> ig = InterpGrid(A, BCreflect, InterpNearest)
4-element InterpGrid{Float64,1,BCreflect,InterpNearest}:
 1.0
 2.0
 3.0
 4.0

julia> y = ig[-3:8]
12-element Array{Float64,1}:
 4.0
 3.0
 2.0
 1.0
 1.0
 2.0
 3.0
 4.0
 4.0
 3.0
 2.0
 1.0

julia> @assert all(abs(y-A[[4:-1:1,1:4,4:-1:1]]) .< EPS)

julia>

Once it runs successfully this way, Grid's tests pass henceforth. However, if I just repeatedly say include("runtests.jl"), then it doesn't get to the point of running correctly.

git bisect identifies 39dfd92 as the first bad commit.

@JeffBezanson JeffBezanson self-assigned this Oct 30, 2014
@JeffBezanson JeffBezanson added the kind:bug Indicates an unexpected problem or unintended behavior label Oct 30, 2014
@JeffBezanson
Copy link
Sponsor Member

cc @JuliaBackports

@timholy
Copy link
Sponsor Member Author

timholy commented Oct 30, 2014

Thanks, @JeffBezanson!

JeffBezanson added a commit that referenced this issue Nov 7, 2014
…verloading

(cherry picked from commit da6087e)

Conflicts:
	test/core.jl
@ivarne
Copy link
Sponsor Member

ivarne commented Nov 7, 2014

Fix backported in f9dcf0d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:bug Indicates an unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

3 participants