-
Notifications
You must be signed in to change notification settings - Fork 316
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
Deviations of this implementation from the original paper #23
Comments
|
Is there a need to redo the routing iterations(in digit capsules) for predict/evaluation/test? |
@nitinsurya In my view, even during the prediction/test, the routing starts with equal probabilities for each primary capsule. So, we need these iterations even during test. what do you think? |
I agree @InnovArul , got a chance to go through routing algorithm again, and realized, these are dynamically weighted, and are dependent on underlying capsule values, which change with each example. |
I have come across the following main deviations between the paper(https://arxiv.org/abs/1710.09829) and the implementation of this repo:
In the repo: it is implemented to be having 8 independent convolution layers with 32 output channels.
reference:
https://github.com/gram-ai/capsule-networks/blob/master/capsule_network.py#L90
https://github.com/gram-ai/capsule-networks/blob/master/capsule_network.py#L67
Please let me know your comments on this.
#24
The text was updated successfully, but these errors were encountered: