OverLordGoldDragon / see-rnn

98%

master: 96%

LAST BUILD ON BRANCH 1.14.4

branch: 1.14.4

CHANGE BRANCH

x

Reset

• 1.14.4
• 1.0
• 1.1
• 1.11
• 1.12
• 1.13
• 1.13.1
• 1.13.2
• 1.13.3
• 1.13.4
• 1.13.5
• 1.14.0
• 1.14.2
• 1.14.3
• 1.14.5
• 1.14.6
• 1.15.0
• 1.15.1
• OverLordGoldDragon-patch-1
• TF2.2
• master
• patch0
• v0.90
• v1
• v1.1
• v1.10
• v1.100
• v1.11
• v1.110
• v1.12
• v1.13
• v1.13.1
• v1.13.2
• v1.14.1
• v1.14.2
• v1.14.3
• v1.14.4

Build # 287

Build Type

push

travis-ci-com

Committed by web-flow

Commit Message

Fix L1-norm case in `weights_norm`

`norm_fn=np.abs` would compute L1 norm as: `np.sqrt(np.sum(np.abs(x)))`, which is incorrect; the sqrt is redundant. `norm_fn=np.abs` will now work correctly. L2-norm case always worked correctly.

For L2-norm, set `norm_fn = (np.sqrt, np.square) = (outer_fn, inner_fn)`, which will compute `outer_fn(sum(inner_fn(x)))`. Note that `norm_fn=np.square` will **no longer compute L2-norm correctly**.

Pardon the mishap.

Run Details

1409 of 1436 relevant lines covered (98.12%)

5.72 hits per line