hughperkins / DeepCL

OpenCL library to train deep convolutional neural networks
Mozilla Public License 2.0
866 stars 200 forks source link

Replacing InputMaker with InputLayerMaker #36

Closed maged closed 8 years ago

maged commented 8 years ago

This line from the NN API:

net->addLayer(InputLayerMaker::instance()->numPlanes(1)->imageSize(28));

Causes the compilation error:

nn.cpp: In function 'int main()':
nn.cpp:11:19: error: incomplete type 'InputMaker' used in nested name specifier
  net->addLayer(InputMaker::instance()->numPlanes(1)->imageSize(28));
               ^

Following the code in the test files though (testsimpleconvolvenet.cpp#L236), and using InputLayerMaker, compilation succeeds

net->addLayer(InputLayerMaker::instance()->numPlanes(1)->imageSize(28));

Should the docs be changed to use InputLayerMaker instead of InputMaker, or is my library linking incorrect?

Thanks

hughperkins commented 8 years ago

Thanks! Addressed in b2a6c58

maged commented 8 years ago

Thanks!