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

Rollout for RLDomain #360

Closed
g-poveda opened this issue May 14, 2024 · 2 comments
Closed

Rollout for RLDomain #360

g-poveda opened this issue May 14, 2024 · 2 comments

Comments

@g-poveda
Copy link
Collaborator

I implemented a pure RLDomain
and just want to run a rollout from it (given an initial state given as input)
however the "domain.set_memory(from_memory)" fails because there is no set_memory for Environment domains such as RL ones, what would be the right way to circumvent this ?

@neo-alex
Copy link
Collaborator

Hello,
The ususal assumption in RL is that you are not necessarily able to initialize the environment from a given state (like in real life) - only an argument-free reset() is generally provided. In your case, you could maybe create a custom Domain inheriting from Simulation (which has a set_memory(...) method) rather than Environment (the one used generally for RL) for the dynamics characteristic.
Does that work out for you?

@g-poveda
Copy link
Collaborator Author

Yes thank you

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