3137
65%
master: 64%

Ran 29 Oct 2018 11:59PM UTC

Jobs 1

Files 244

Run time 18s

Badge

Embed ▾

pending completion

Build # 3137

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

Jobs

ID	Job ID	Ran	Files	Coverage
1	3137.1	29 Oct 2018 11:59PM UTC	0	64.34	Travis Job 3137.1

Source Files on build 3137

Detailed source file information is not available for this build.

rlworkgroup / garage / 3137
65%
master: 64%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3137

rlworkgroup / garage / 3137 65% master: 64%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3137

rlworkgroup / garage / 3137
65%
master: 64%

README BADGES
x