sawcordwell / pymdptoolbox

Markov Decision Process (MDP) Toolbox for Python
BSD 3-Clause "New" or "Revised" License
518 stars 252 forks source link

fixed firemdp.py AssertionError caused by incorrect argument sequence #12

Closed thejat closed 9 years ago

thejat commented 9 years ago

This is a very minor fix but helps with new users who want to run examples to see what the module does.

coveralls commented 9 years ago

Coverage Status

Coverage remained the same at 87.12% when pulling f32a6d86d06f9ac60724b738e60cc1629796f722 on thejat:example_fix into f094638f9522ab1a9ef182c9e58b0099566eb8fb on sawcordwell:master.

sawcordwell commented 9 years ago

Thanks for the fix, it now finishes executing!

However, that file is example is still a work in progress which I unfortunately did not document. I have not finished implementing getTransitionProbabilities for actions 2 and 3... contributions welcome.