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
Including sparse convolutions inside traditional model #26
Comments
Using sparse convolutions only makes sense if the input is spatially sparse. What is your input? The output of dense convolutions will not be sparse, so you should not use dense convolutions followed by sparse convolutions. |
Hi, thanks for your reply. |
That might work. But the learning signal is the dense layers will be limited to the sites that are not filtered out, which could be problematic. |
I think that this should not be an issue, correct me if I'm wrong. I'll give you some more information about my network, so that you could better understand what I'm trying to do. |
Could I simply implement a layer where I map my input to an InputBatch (in the updateOutput function), like you do in your example, and the sparse gradOutput to a dense gradInput (in the updateGradInput)? |
Please contact me at btgraham@gmail.com so we can discuss this in more detail. |
Hi, first of all thank you so much for sharing this amazing library, I'm willing to use it in my MSc thesis.
I have a few questions: If I understood everything correctly, you advise to create a network and forward it an InputBatch created manually. This works well for a network where you have only sparse convolutions. Unfortunately, I have a network where I use a large majority of dense convolutions, and I would like to add some sparse convolutive layers at the end of it. Is it possible to achieve this?
I saw that in the pytorch version of the library there is a DenseToSparse layer that would (probably) solve this issue, but you advise not to use it. What is the reason? Could I port it to Lua, or do you plan to implement it in Lua?
Thanks in advance for your time, and I'm sorry if I misunderstood something.
The text was updated successfully, but these errors were encountered: