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.

Sign up for GitHub

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

Jump to bottom

Simplify PER sample idx #281

Merged

lgraesser merged 4 commits into master from per-mod-sample

Feb 6, 2019

Collaborator

lgraesser commented Feb 6, 2019

Fix

Simplify how indices are sampled from the SumTree in PER. It eliminates binning the trees range into batch_size segments and sampling a value from each segment. Instead each target value is sampled from the entire range of the tree.

lgraesser and others added 4 commits

February 5, 2019 21:58


          Simplify PER sample idx

e08e5c0


          Merge branch 'master' into per-mod-sample

78c165f


          Updating unit test to reflect new sampling behaviour

b3a3543


          Merge branch 'per-mod-sample' of https://github.com/kengz/SLM-Lab int…

9b4c25c

…o per-mod-sample

lgraesser merged commit a716a0d into master

kengz deleted the per-mod-sample branch

February 6, 2019 16:41

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet