JuliaPOMDP / DiscreteValueIteration.jl

Value iteration solver for MDPs
Other
20 stars 12 forks source link

some problem with terminal states in the sparse solver #26

Closed zsunberg closed 5 years ago

zsunberg commented 5 years ago

I tried the sparse solver with UnderlyingMDP(SubHuntPOMDP()) and it gave

VI: Error During Test at /home/zach/.julia/dev/SubHunt/test/runtests.jl:12
  Got exception outside of a @test
  BoundsError: attempt to access 4-element Array{StaticArrays.SArray{Tuple{2},Int64,1,2},1} at index [-1]
  Stacktrace:
   [1] getindex at ./array.jl:731 [inlined]
   [2] transition(::SubHuntPOMDP, ::SubState, ::Int64) at /home/zach/.julia/dev/SubHunt/src/transition.jl:15
   [3] transition at /home/zach/.julia/dev/POMDPModelTools/src/underlying_mdp.jl:15 [inlined]
   [4] reward_s_a(::UnderlyingMDP{SubHuntPOMDP,SubState,Int64}) at /home/zach/.julia/dev/DiscreteValueIteration/src/sparse.jl:83
   [5] solve(::SparseValueIterationSolver, ::UnderlyingMDP{SubHuntPOMDP,SubState,Int64}) at /home/zach/.julia/dev/DiscreteValueIteration/src/sparse.jl:107
   [6] macro expansion at /home/zach/.julia/dev/SubHunt/test/runtests.jl:18 [inlined]
   [7] macro expansion at /buildworker/worker/package_linux64/build/usr/share/julia/stdlib/v1.0/Test/src/Test.jl:1083 [inlined]
   [8] top-level scope at /home/zach/.julia/dev/SubHunt/test/runtests.jl:13
   [9] include at ./boot.jl:317 [inlined]
   [10] include_relative(::Module, ::String) at ./loading.jl:1044
   [11] include(::Module, ::String) at ./sysimg.jl:29
   [12] include(::String) at ./client.jl:392
   [13] top-level scope at none:0
   [14] eval(::Module, ::Any) at ./boot.jl:319
   [15] exec_options(::Base.JLOptions) at ./client.jl:243
   [16] _start() at ./client.jl:425

Will need to track this down.