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

SparseTabularMDP doesn't support initialstate #444

Closed
fredcallaway opened this issue Nov 4, 2022 · 3 comments
Closed

SparseTabularMDP doesn't support initialstate #444

fredcallaway opened this issue Nov 4, 2022 · 3 comments

Comments

@fredcallaway
Copy link
Contributor

If I define an MDP with QuickMDP, I can specify an initialstate distribution which is automatically used in e.g. RolloutSimulator. But when I convert the QuickMDP to a SparseTabularMDP (for faster rollouts) the initialstate is lost. Any particular reason for this?

@zsunberg
Copy link
Member

zsunberg commented Nov 4, 2022

Upon looking, I see that SparseTabularMDP does not even have a field for representing the initial distribution.

@zsunberg
Copy link
Member

zsunberg commented Nov 4, 2022

Someone should make a pull request to add this. The probable reason that this was not added was that usually the sparse tabular representation is used for value iteration or similar offline methods that do not need an initial state distribution. But someone should add it for simulation purposes

@fredcallaway
Copy link
Contributor Author

Cool, I'm on it

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

No branches or pull requests

2 participants