Skip to content
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

Support current version of mxnet #11

Open
leezu opened this issue Dec 6, 2016 · 1 comment
Open

Support current version of mxnet #11

leezu opened this issue Dec 6, 2016 · 1 comment

Comments

@leezu
Copy link

leezu commented Dec 6, 2016

It seems mxnet.js does not compile with current versions of mxnet.

Can you give some suggestions where to pick up the work to bring mxnet.js up to date with the current mxnet? Will the compatibility be affected by the (as far as I understand soonish) merge of nnvm?

@leezu
Copy link
Author

leezu commented Dec 7, 2016

The error lies within the amalgamation code. This has been fixed with apache/mxnet#4135 and I have confirmed that the mxnet.js image classification demo works with a recompiled version of libmxnet_predict.js (based on apache/mxnet@c05efbe ).

I noticed that the compilation will only work with a recent version of emscripten (at least it fails for me with the distro provided version of Ubuntu 16.04).

Would you be willing to accept a pull request that updates mxnet to above mentioned commit and adds a notice about the emscripten version?

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

No branches or pull requests

1 participant