`FullyObservablePOMDP` `obsindex` bugfix

JuliaPOMDP / POMDPs.jl

MDPs and POMDPs in Julia - An interface for defining, solving, and simulating fully and partially observable Markov decision processes on discrete and continuous spaces.

http://juliapomdp.github.io/POMDPs.jl/latest/

Other

657 stars 100 forks source link

`FullyObservablePOMDP` `obsindex` bugfix #485

Closed WhiffleFish closed 1 year ago

WhiffleFish commented 1 year ago

Removed unnecessary parametric typing for obsindex + added test.

Also removed redundant observation function for FullyObservablePOMDP that is already covered in POMDPs.jl.

https://github.com/JuliaPOMDP/POMDPs.jl/blob/cc15fe957d97ec1c41a2415843eebe8157670ab1/lib/POMDPTools/src/ModelTools/fully_observable_pomdp.jl#L22-L28

(Lines 26-28 redundant)

https://github.com/JuliaPOMDP/POMDPs.jl/blob/cc15fe957d97ec1c41a2415843eebe8157670ab1/src/pomdp.jl#L64

zsunberg commented 1 year ago

Thanks @WhiffleFish !