vitamin_c icon indicating copy to clipboard operation
vitamin_c copied to clipboard

r1 weights and biases naming redundancy

Open chrismessenger opened this issue 4 years ago • 2 comments

The convolutional weight and bias names have historical + '1' added to them which is not required. This is also the case in the q and r2 networks.

Plus, there are lots of commented out code that can get deleted when creating the weights.

chrismessenger avatar Nov 21 '20 09:11 chrismessenger

Also, the

self.nonlinearity_mean = tf.clip_by_value

is never used so can be removed.

chrismessenger avatar Nov 21 '20 09:11 chrismessenger

You can also delete the commented out "make positive definite covariance matrix code"

chrismessenger avatar Nov 21 '20 09:11 chrismessenger