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

Broken test convert error #690

Closed
mforets opened this issue Oct 4, 2019 · 3 comments · Fixed by #695
Closed

Broken test convert error #690

mforets opened this issue Oct 4, 2019 · 3 comments · Fixed by #695
Labels

Comments

@mforets
Copy link
Member

mforets commented Oct 4, 2019

Computing successors 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████| Time: 0:00:02
Reachability.solve_continuous: Error During Test at /home/mforets/.julia/dev/Reachability/test/Reachability/alltests.jl:2
  Got exception outside of a @test
  LoadError: MethodError: Cannot `convert` an object of type ReachSet{Hyperrectangle{Float64,Array{Float64,1},Array{Float64,1}}} to an object of type ReachSet{Hyperrectangle{Float64,VNC,VNR} where VNR<:AbstractArray{Float64,1} where VNC<:AbstractArray{Float64,1}}
  Closest candidates are:
    convert(::Type{#s16} where #s16<:Union{Number, T}, !Matched::MultivariatePolynomials.AbstractPolynomialLike{T}) where T at /home/mforets/.julia/packages/MultivariatePolynomials/0oEYT/src/conversion.jl:15
    convert(::Type{T}, !Matched::T) where T at essentials.jl:167
    ReachSet{Hyperrectangle{Float64,VNC,VNR} where VNR<:AbstractArray{Float64,1} where VNC<:AbstractArray{Float64,1}}(::Any, !Matched::Any, !Matched::Any) where SN at /home/mforets/.julia/dev/Reachability/src/ReachSets/ReachSet.jl:16
  Stacktrace:
   [1] setindex!(::Array{ReachSet{Hyperrectangle{Float64,VNC,VNR} where VNR<:AbstractArray{Float64,1} where VNC<:AbstractArray{Float64,1}},1}, ::ReachSet{Hyperrectangle{Float64,Array{Float64,1},Array{Float64,1}}}, ::Int64) at ./array.jl:766
   [2] project_reach(::Array{SparseReachSet{CartesianProductArray{Float64,LazySet{Float64}}},1}, ::Array{Int64,1}, ::Options) at /home/mforets/.julia/dev/Reachability/src/ReachSets/project_reach.jl:95
   [3] project(::Array{SparseReachSet{CartesianProductArray{Float64,LazySet{Float64}}},1}, ::Options) at /home/mforets/.julia/dev/Reachability/src/ReachSets/project_reach.jl:117
   [4] project(::ReachSolution{CartesianProductArray{Float64,LazySet{Float64}},SparseReachSet{CartesianProductArray{Float64,LazySet{Float64}}}}) at /home/mforets/.julia/dev/Reachability/src/ReachSets/project_reach.jl:120
   [5] top-level scope at /home/mforets/.julia/dev/Reachability/test/Reachability/solve_continuous.jl:37
   [6] include at ./boot.jl:328 [inlined]
   [7] include_relative(::Module, ::String) at ./loading.jl:1094
   [8] include(::Module, ::String) at ./Base.jl:31
   [9] include(::String) at ./client.jl:431
   [10] top-level scope at /home/mforets/.julia/dev/Reachability/test/Reachability/alltests.jl:2
   [11] top-level scope at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.2/Test/src/Test.jl:1113
   [12] top-level scope at /home/mforets/.julia/dev/Reachability/test/Reachability/alltests.jl:2
   [13] top-level scope at util.jl:156
   [14] include at ./boot.jl:328 [inlined]
   [15] include_relative(::Module, ::String) at ./loading.jl:1094
   [16] include(::Module, ::String) at ./Base.jl:31
   [17] include(::String) at ./client.jl:431
   [18] top-level scope at /home/mforets/.julia/dev/Reachability/test/runtests.jl:9
   [19] include at ./boot.jl:328 [inlined]
   [20] include_relative(::Module, ::String) at ./loading.jl:1094
   [21] include(::Module, ::String) at ./Base.jl:31
   [22] exec_options(::Base.JLOptions) at ./client.jl:295
   [23] _start() at ./client.jl:464
  in expression starting at /home/mforets/.julia/dev/Reachability/test/Reachability/solve_continuous.jl:37
  
Test Summary:                 | Pass  Error  Total
Reachability.solve_continuous |    4      1      5
ERROR: LoadError: LoadError: Some tests did not pass: 4 passed, 0 failed, 1 errored, 0 broken.
@mforets
Copy link
Member Author

mforets commented Oct 4, 2019

This error is allowed in the travis build for this package, as it was introduced in the dev version of LazySets.

@schillic
Copy link
Member

schillic commented Oct 4, 2019

Okay, and is it a valid error in LazySets? Then we should fix it there before we make a new release.

@schillic
Copy link
Member

schillic commented Oct 4, 2019

I investigated and am pretty sure that the problem occurred by adding new type parameters to Hyperrectangle. But the actual problem lies in Reachability:

RsetsProj[i] = ReachSet(oa(projected), t0, t1)

So it seems that our workaround has already found its limits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants