Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

r1 weights and biases naming redundancy #87

Open
chrismessenger opened this issue Nov 21, 2020 · 2 comments
Open

r1 weights and biases naming redundancy #87

chrismessenger opened this issue Nov 21, 2020 · 2 comments
Assignees

Comments

@chrismessenger
Copy link
Collaborator

chrismessenger commented Nov 21, 2020

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
Copy link
Collaborator Author

Also, the

self.nonlinearity_mean = tf.clip_by_value

is never used so can be removed.

@chrismessenger
Copy link
Collaborator Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants