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

Jump to bottom

Policy Representation #5

Closed
markkho opened this issue Nov 2, 2020 · 2 comments
Closed

Policy Representation #5

markkho opened this issue Nov 2, 2020 · 2 comments

Comments

@markkho
Copy link
Owner

markkho commented Nov 2, 2020

How should we revamp the policy representation to be more generic? One idea is to make it more like a numpy array where you "index" things by state or action.
@cgc
Copy link
Collaborator

cgc commented Mar 30, 2021

definitely think making the most of python's syntax to make policy access simpler/easier makes a lot of sense. I think __getitem__ (policy[s]) is a cool solution. __call__ is another option, but seems less ideal in way that I'm struggling to articulate.

@markkho
Copy link
Owner Author

markkho commented Apr 5, 2021

TabularPolicy extends dict now, so this is resolved in 554bcd2

@markkho markkho closed this as completed Apr 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@cgc @markkho