-
Notifications
You must be signed in to change notification settings - Fork 69
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
assert total_steps >= num_timesteps #52
Comments
Could you print the value for total_steps? |
Sorry I have already restart.I can take a test and report details. |
Yeah thx, it will be helpful if you could reproduce it. |
Sorry for this late reply. I finally reproduce this problem.
this is all time steps:
I think you are right . |
Thanks for the additional information. Indeed it seems to be an overflow problem. There's a suggestion of fix in google/brax#578. I'll give it a try and update it here if any. |
Hello, I recently meet a problem as follows:
I think it maybe relate with ppo_params, this is my ppo params:
The text was updated successfully, but these errors were encountered: