Generating Handwritten Digits on MNIST using GANs#161
Generating Handwritten Digits on MNIST using GANs#161swaingotnochill wants to merge 7 commits intomlpack:masterfrom
Conversation
|
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
|
@zoq @kartikdutt18 When I am adding convulation layer, it seems to give an error that
Is it due to syntax error while using the modules or something else? I tried working around it, but don't know why its giving push_back not found(this function should be available). your thoughts? |
| @@ -0,0 +1,404 @@ | |||
| { | |||
There was a problem hiding this comment.
model.Add(new mlpack::ann::Convolution<>(,6,5,5,1,1,0,0,28,28))
Could you try this?
Reply via ReviewNB
There was a problem hiding this comment.
I tried but it doesn't seems to work too.
input_line_65:2:16: error: no matching constructor for initialization of 'mlpack::ann::Convolution<>' model.Add(new mlpack::ann::Convolution<>(1,6,5,5,1,1,0,0,28,28)); ^ ~~~~~~~~~~~~~~~~~~~~~ /home/viole/anaconda3/envs/notebook/include/mlpack/methods/ann/layer/layer_types.hpp:172:7: note: candidate constructor (the implicit copy constructor) not viable: requires 1 argument, but 10 were provided class Convolution; ^ /home/viole/anaconda3/envs/notebook/include/mlpack/methods/ann/layer/layer_types.hpp:172:7: note: candidate constructor (the implicit move constructor) not viable: requires 1 argument, but 10 were provided
No matching constructor..
There was a problem hiding this comment.
I just tested it on lab.mlpack.org and there it works fine, I had some problems with namespaces in the past, can you put each namespace into a separate cell?
b354d5a to
4685f83
Compare
zoq
left a comment
There was a problem hiding this comment.
For an example about how to save/load a model you can checkout https://github.com/mlpack/examples/blob/master/mnist_simple/mnist_simple.cpp, the interesting part is:
https://github.com/mlpack/examples/blob/master/mnist_simple/mnist_simple.cpp#L170
and
examples/mnist_simple/mnist_simple.cpp
Line 174 in 776e116
Let me know if I should clarify anything further.
|
Hi Swain, Could you push the latest changes (if you have got the issue resolved). |
Sure...I will push it by Tonight (I don't have the computer right now ) |
|
I redirected the PR at #172 . Closing this one. |
This PR is for discussing the issues and working on GANs notebook for MNIST dataset.