Skip to content

Hotfix v0.9.1 - develop #2421

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

Merged
merged 11 commits into from
Aug 10, 2019
Merged

Hotfix v0.9.1 - develop #2421

merged 11 commits into from
Aug 10, 2019

Conversation

ervteng
Copy link
Contributor

@ervteng ervteng commented Aug 10, 2019

  • Fix issue with BC Trainer increment_steps.
  • Fix issue with Demonstration Recorder and visual observations (memory leak fix was deleting vis obs too early).
  • Make Samplers sample from the same random seed every time, so generalization runs are repeatable.
  • Fix crash when using GAIL, Curiosity, and visual observations together.
  • Note: Only incremented mlagents package version as the others shouldn't have changed.

There is a conflict. I will resolve the conflict on this branch after the hotfix is merged to master.

Ervin T and others added 11 commits August 1, 2019 15:24
#2398)

* Change samplers to use random state to allow consistency in reset parameter draws for a specified seed
- Fix issue with BC Trainer `increment_steps`. 
- Fix issue with Demonstration Recorder and visual observations (memory leak fix was deleting vis obs too early). 
- Make Samplers sample from the same random seed every time, so generalization runs are repeatable.
- Fix crash when using GAIL, Curiosity, and visual observations together.
@ervteng ervteng requested a review from xiaomaogy August 10, 2019 01:01
@ervteng ervteng merged commit 84a9322 into develop Aug 10, 2019
@ervteng ervteng deleted the hotfix-v0.9.1 branch August 10, 2019 01:11
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants