Closed zcczhang closed 3 years ago
This pull request introduces 18 alerts when merging 18a765d154c67829ec3d1dfb1a6f1493425da410 into e3ba00faeb6114777c8c0b3da2fd6a752fcc1a55 - view on LGTM.com
new alerts:
This pull request introduces 1 alert when merging 4c083629b9529aef0c8c8635af427d4a06c45689 into e3ba00faeb6114777c8c0b3da2fd6a752fcc1a55 - view on LGTM.com
new alerts:
Nice updates! Looks good to merge to me.
Hi, here are what I added/modified:
GymMuJoCoSensor
class ingym_sensors.py
GymContinuousMuJoCoTask
class ingym_task.py
. However, as this is the same asGymContinuousBox2D
, it can be combined into one method in the future, something like aGymContinuousTask
methodtask_selector()
andsensor_selector()
ingym_tasks.py
gym_baselines
inproject
. Some notes:gym_model.py
ingym_baselines/models
. To clarify, I added this file only just for the format consistency with other baselines in the project, so it is just the same asallenact_plugins.gym_models.py
so far.gym_mujoco_tutorail
intutorials
. Now there are only codes there and I will modify it to a literate experiment file with proper syntax later in case somewhere need to be changed after reviewing.bash scripts/auto_format.sh