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
SAC HER example results not matching #49
Comments
Can you run multiple seeds? I included 5 separate runs since it does tend to fail sometimes. |
Sorry, but I can't seem to find the right place to set the seeds. Would you link me to a line number? |
It should be set randomly so you can just run the script again.
…On Mon, Apr 22, 2019, 2:49 PM George Joseph ***@***.***> wrote:
Sorry, but I can't seem to find the right place to set the seeds. Would
you link me to a line number?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#49 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AAJ4VZOCBU2VVJOXJZJ5JZDPRYXIJANCNFSM4HHQCCRA>
.
|
On a side note, I was trying FetchPush-v1. The only changes I need to make is in gym.make(... ) right? If so, I've tried multiple seeds with about 500 iterations, with no progress. Am I missing something? What's the expected time? |
Based on this paper, it seems like you need one millions steps, so probably (1000 iterations). But they also use DDPG with an epsilon greedy exploration policy, so that might matter. |
I cloned the repo, setup the environment and ran (made no changes)
python her_sac_gym_fetch_reach.py
The results don't seem to match with this. Did something break in the latest commit?
However, when I try the td3, it works fine
python her_td3_multiworld_sawyer_reach.py
The text was updated successfully, but these errors were encountered: