1
65%
master: 64%

Ran 29 Oct 2018 11:59PM UTC

Files 244

Run time 17s

Badge

Embed ▾

Committed 29 Oct 2018 10:54PM UTC coverage: 64.341%. First build

Job # 3137.1

Build Type

push

travis-ci-com

Committed by

naeioi

Commit Message

Fix TensorFlow warning in NPO about IndexedSlices

NPO uses a boolean valid_var tensor to select a slice from a variety
of tensors related to action, policy and advantages. Previously we
used tf.boolean_mask() to do the selection, which internally calls
tf.gather() that emits tf.IndexedSlices. When calculating BP,
tf.IndexedSlices will be casted into a dense tf.Tensor with a
"Converting sparse IndexedSlices to a dense Tensor of unknown shape"
warning. This fix replaces tf.boolean_mask() with
tf.dynamic_partition(), which don't have such problem.

This difference of behavior is not documented in Tensorflow. It is
not clear if tf.dynamic_partition() will cause performance
regression.

This fixes #289.

Run Details

9648 of 14995 relevant lines covered (64.34%)

0.64 hits per line

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

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3137.1

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

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3137.1

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

README BADGES
x