-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Make Continuous Mountain Car Environment same as Gym implementation #1394
Conversation
@@ -96,13 +96,17 @@ class ContinuousMountainCar | |||
* @param velocityMax Maximum legal velocity. | |||
*/ | |||
ContinuousMountainCar(const double positionMin = -1.2, | |||
const double positionMax = 0.5, | |||
const double positionMax = 0.6, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a comment for each new parameter? Also I think you did some good additions to this file, so feel free to add yourself as another author.
return 100.0; | ||
return -pow(action.action[0], 2)*0.1; | ||
reward = 100.0; | ||
reward -= std::pow(action.action[0], 2) * 0.1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't think this makes a huge difference, but I guess it's a good idea to get the same results as the gym env.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No comments from my side; thanks for the contribution! :)
@mlpack-jenkins test this please |
Thanks again for the contribution 👍 |
This PR make the contiunous Mountain Car Environment streamlined with the implementation in OpenAI Gym (https://github.com/openai/gym/blob/master/gym/envs/classic_control/continuous_mountain_car.py) . The changes are :
I have also added my name to list of contributors following #1388