rlworkgroup / garage / 2894 / 1
65%
master: 64%

LAST BUILD BRANCH: release-2018.10
DEFAULT BRANCH: master
Ran
Files 242
Run time 22s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 63.22% (+0.4%) from 62.796%
2894.1

Pull #329

travis-ci-com

web-flow 
Add miscellaneous changes

 - Make the replay buffer base class garage.replay_buffer.ReplayBuffer
abstract. Rename the regular replay buffer to SimpleReplayBuffer.
 - Delete enums and pass ReplayBuffer class into algorithm. Now use can
construct a replay buffer by their own.
 - Move `configure_dims` and `dims_to_shapes` into replay buffer
constructor. Delete `env.reset()` in configure_dims. Notice the reason
that we need to use `env.step()` because it's the only way to get
env_info in gym.GoalEnv. And the env_info is used to compute reward in
`env.compute_reward`. But the source code in gym shows that the env_info
is not actually used when computing reward. So we can not store info in
transitions.
 - Set default value for optimizer params in DDPG.
 - Remove commented code.
 - Redesign OffPolicyBatchPolopt. The BatchPolopt is more compatible for
on policy algorithms. In last commit, OffPolicyBatchPolopt inherits from
it. But actually off policy algorithms usually samples from experience
replay instead of batch samples optimization. So it sounds like an
unusual name. So rename it to OffPolicyAlgorithm and inherits from
garage.algos.RLAlgorithm. Also, the training process in
OffPolicyBatchPolopt is more specific for DDPG. Before design a sound
training process function, I move the training function into DDPG.
Pull Request #329: Fix VPG and DDPG to use VectorizedSampler

9483 of 15000 relevant lines covered (63.22%)

0.63 hits per line

Source Files on job 2894.1