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

Misleading comment in MDP class #23

Closed
GoogleCodeExporter opened this issue Oct 21, 2015 · 1 comment
Closed

Misleading comment in MDP class #23

GoogleCodeExporter opened this issue Oct 21, 2015 · 1 comment

Comments

@GoogleCodeExporter
Copy link

The following definition is present in the MDP class:

   def T(self,state, action): 
        """Transition model.  From a state and an action, return a list
        of (result-state, probability) pairs."""

the value iteration procedure, though, expects the transition model to return a 
list of (probability,result-state) pairs. I haven't looked at what other 
algorithms might be expecting.

Original issue reported on code.google.com by Gustavo....@gmail.com on 21 Apr 2011 at 6:26

@GoogleCodeExporter
Copy link
Author

Fixed in r42.

Original comment by wit...@gmail.com on 31 Aug 2011 at 5:19

  • Changed state: Fixed

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

No branches or pull requests

1 participant