-
Notifications
You must be signed in to change notification settings - Fork 7
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
Issues with code example (CNN example and example in README) #6
Comments
Hi @yanivnik. I also encounter the same error messages from running the two examples (visualize_cnn_random.py and visualize_mlp_random.py). I am just wondering if you have had any luck solving the issue? |
@ShaneXLi I played around with some constants in the code to make it run, but unfortunately the resulting visualizations are still not very similar to the expected results presented in the paper. @AhmedImtiazPrio could you have a look at these demos? |
Hi Ahmed. As the error suggested, the two matrices cannot be multiplied due to their matrix shapes. So this error happens consistently every time I run the current/latest version of example codes. Please take some time to review the codes. |
It looks like an issue raised due to some changes that have been made to the splinecam library. I'll update the examples and ping here. |
Thank you for publishing the code. The method is interesting and seems like it could be helpful to me on a current research project.
For that reason I tried running the code examples, but I'm having some trouble getting them work, mainly around the shape of the projection matrix.
I tried running the "visualize_cnn_random.py" example (without any edit to the code), and I get the following error -
The input to the function
x
is with shape(1,2)
, the shape of the projection matrixT
(andself.Abw
) is(3072,2)
.It is unclear to me why the index is only accessed until the last element (
self.Abw[...**,:-1**]
) there. Could this be the cause of the issue?Also, I looked at the code example in the README and tried using it, passing the
domain
parameter it as a list of 2D indices (As described in the paper and in one of the example notebooks), but it fails in another unclear exception -Is this an issue with the code? Or should the
domain
parameter be passed as something other than a list of 2D points? If so, could you please clarify this in the README?Thank you so much for your assistance!
The text was updated successfully, but these errors were encountered: