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

Fix/deterministic action space sampling #1103

Conversation

maxhuettenrauch
Copy link
Collaborator

  • I have added the correct label(s) to this Pull Request or linked the relevant issue(s)
  • I have provided a description of the changes in this Pull Request
  • I have added documentation for my changes
  • If applicable, I have added tests to cover my changes.
  • I have reformatted the code using poe format
  • I have checked style and types with poe lint and poe type-check
  • (Optional) I ran tests locally with poe test
    (or a subset of them with poe test-reduced) ,and they pass
  • (Optional) I have tested that documentation builds correctly with poe doc-build

This PR fixes action space seeding in sub process based vectorized environments.

@maxhuettenrauch
Copy link
Collaborator Author

Addresses #1083

@MischaPanch MischaPanch marked this pull request as ready for review April 18, 2024 11:51
@MischaPanch MischaPanch merged commit a043711 into thu-ml:master Apr 18, 2024
4 checks passed
@MischaPanch MischaPanch deleted the fix/deterministic-action-space-sampling branch April 18, 2024 14:17
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

Successfully merging this pull request may close these issues.

None yet

2 participants