Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Fix predict c api #4537

Merged
merged 3 commits into from Jan 6, 2017
Merged

Fix predict c api #4537

merged 3 commits into from Jan 6, 2017

Conversation

howard0su
Copy link
Contributor

Fix the regression in predict_api. While I am here, update and cleanup the C++ image classification example.


image-classification-predict.o: image-classification-predict.cc
g++ -O3 -c image-classification-predict.cc ${CFLAGS}
g++ -O0 -c image-classification-predict.cc ${CFLAGS}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why change this?

@howard0su
Copy link
Contributor Author

howard0su commented Jan 6, 2017 via email

@songtianyi
Copy link
Contributor

This pr haven't been merged yet? Waiting merging to test.

@piiswrong piiswrong merged commit b67a445 into apache:master Jan 6, 2017
rravu3 pushed a commit to rravu3/mxnet that referenced this pull request Jan 21, 2017
* Fix predict c api

* Update comm.h
@Piyush3dB
Copy link
Contributor

@piiswrong I get the following errors when I load symbol from JSON file using the _LIB.MXSymbolCreateFromFile() function

[14:35:44] mxnet/dmlc-core/include/dmlc/./logging.h:300: [14:35:44] src/core/op.cc:55: Check failed: op != nullptr Operator Convolution is not registered
[14:47:36] mxnet/dmlc-core/include/dmlc/./logging.h:300: [14:47:36] src/core/op.cc:55: Check failed: op != nullptr Operator FullyConnected is not registered

Is this a known issue or could I be doing something wrong?

If I construct the same symbol using mx.symbol.* objects then I don't get any error when I execute the symbol.

@Piyush3dB
Copy link
Contributor

To get rid of the errors I'm seeing above, I have to first call the function _LIB.MXListAllOpNames() before calling _LIB.MXSymbolCreateFromFile(). All is good and working for me now.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants