sawcordwell / pymdptoolbox

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

Add skip_check argument in MDP and subclasses #11

Closed yasserglez closed 9 years ago

yasserglez commented 9 years ago

I've added a skip_check argument (set to False by default) in MDP and all subclasses. I think it might be useful to skip checking the transition and reward matrices on the creation of every MDP instance if one knows that they are correct beforehand (which can be verified using mdptoolbox.util.check). For example, this can save some time when the same MDP is solved using different algorithms.

coveralls commented 9 years ago

Coverage Status

Coverage increased (+0.04%) to 87.12% when pulling ffaef478fd3ad63b355b0d6a17ee0c7541a650b2 on yasserglez:skip_check into 8fac9a11e0792606eef15b1df450307e7f9d6a8c on sawcordwell:master.

sawcordwell commented 9 years ago

Sounds good. The Travis build error is just that pip couldn't download cvxopt on one of the tests, but the others all pass so I'm confident that it is working as expected.