In the current code, spectral normalization does not seem to be properly applied

[dlqudwns]

When adding FC layer, BNN class adds copy of layer, according to

https://github.com/tianheyu927/mopo/blob/master/mopo/models/bnn.py#L141

Further, when copying a layer, the code uses repr function:
https://github.com/tianheyu927/mopo/blob/master/mopo/models/fc.py#L132
https://github.com/tianheyu927/mopo/blob/master/mopo/models/fc.py#L51

However, the sn attribute is not specified in repr, and it uses default value sn=False.

Even if I add sn in the repr, tensorflow gives error due to the redundant name 'u'.

[tianheyu927]

Hi ByungJun,

Thanks for catching that! Spectral normalization is indeed not properly applied in the code. We have updated the code to remove the spectral normalization and will update the paper to exclude the discussion on using Lipschitz regularization in the practical algorithm as soon as possible. Note that Lipschitz regularization on the model is mainly a regularization heuristic and removing it won’t affect the soundness of the paper. Sorry for the confusion!