Closed xinyutan17 closed 8 years ago
Okay fixed, can I merge now?
Yes, though people should be aware that this breaks any of their non-checked in hyperparams files (and they need to change those two lines of code with the directory)
Should we just keep the old version then? I mean the extra "../" is kind of stupid but trivial.
Is there a clean way to make this change without affecting the hyperparams file?
Yes, those changes don't really do anything except make the code look a little better, so I can just remove them.
Ok, please do then.
On Sun, Apr 17, 2016 at 4:20 PM, Xin Yu Tan notifications@github.com wrote:
Yes, those changes don't really do anything except make the code look a little better, so I can just remove them.
— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/cbfinn/gps/pull/13#issuecomment-211127659
Run gps from any directory (except MuJoCo experiments)