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.

Sign up for GitHub

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

Smart GridWorld Sim takes component wrappers #420

Open
rusu24edward opened this issue Aug 8, 2023 · 0 comments
Open

Smart GridWorld Sim takes component wrappers #420

rusu24edward opened this issue Aug 8, 2023 · 0 comments
Labels
enhancement New feature or request
Milestone
Coming... eventually

Comments

@rusu24edward
Copy link
Collaborator

#72 added support for state, observers, dones, and a component registry. We should add support for component wrappers. Two things to consider:

  1. For smart support via text and the registry, we could interpret the input, like so "RavelActionWrapper(MoveActor)", or "RavelActionWrapper | Move Actor", or something like that.
  2. For smart support via the class instance... this is not so clear. Currently the components get passed as a set. We could pass it as a tree structure, where everything underneath a particular wrapper in the structure would be wrapped by that wrapper.
@rusu24edward rusu24edward added the enhancement New feature or request label Aug 8, 2023
@rusu24edward rusu24edward added this to the Coming... eventually milestone Aug 8, 2023
This was referenced Aug 8, 2023
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
Coming... eventually
Development

No branches or pull requests
1 participant
@rusu24edward