-
Notifications
You must be signed in to change notification settings - Fork 122
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Same seed different results #34
Comments
Hello! Thanks for mentioning this! Can you check if you get the same issue on the pybullet envs as well? Since the reset of the state is handled directly by pybullet through its loading and saving mechanism, I do not have any influence on the deterministic execution of different runs. |
Just tested. To have similar results among different runs, there is the need to set the seed also for the action space and observation space:
To have the same results among different resets, the seed needs to be reset everytime. This script:
returns:
This happens with whatever environment I tested. |
If you reset the seed in my envs every time, does that help too? If so, then we fix this to be stored across resets. |
Yes, I tried to set the seed before every reset with different environments from the repo and it seems to work consistently. I also think that the .seed() method should set not only the env seed, but also the action_space and the observation_space ones. At least to give consistency, given that in other Gym environments the only function I had to call to set the seed was .seed(). |
Ok, so we make that functionality consistent with mujoco and make it store
the seed across resets.
…On Thu, Nov 21, 2019, 15:11 Giuseppe Paolo ***@***.***> wrote:
Yes, I tried to set the seed before every reset with different
environments from the repo and it seems to work consistently.
I also think that the .seed() method should set not only the env seed, but
also the action_space and the observation_space ones. At least to give
consistency, given that in other Gym environments the only function I had
to call to set the seed was .seed().
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#34?email_source=notifications&email_token=AAXXXK7K5CZ55OLKZPPTYF3QU2JKXA5CNFSM4JNLEHI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE2LGKY#issuecomment-557101867>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAXXXKZQX4MBRBFSWWADJTDQU2JKXANCNFSM4JNLEHIQ>
.
|
Hi,
If I set the seed, I get different results between different runs.
I am attaching a small script to reproduce the issue:
The results I obtain are:
And they change among any run.
I think this is a bug, cause the result should be always the same, given the same seed.
Or am I doing something wrong?
The text was updated successfully, but these errors were encountered: