-
Notifications
You must be signed in to change notification settings - Fork 79
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
merging into mxnet repo #71
Comments
@lx75249 @zhangchen-qinyinghua |
The most convinient way is to add a subdirectory (cpp-package) in mxnet's root, like other language bindings (R and scala). In this way, the main work is test integration. (I guess we can also add some hook for regenerating op.h when mxnet is updated? I'm not sure if it's possible in github) |
I agree with the subdirectory approach. As for the update hook, I guess we can hook it, but not sure if we can execute python script inside the hook. |
OK. I will do it. |
I think this should be done ASAP. Because this repo is far behind other interfaces, e.g, Python. Meanwhile, it is really important for industrial envo (e.g, data center), where Python and other dependencies are normally not installed. |
I agree :) |
I propose to merge mxnet.cpp into mxnet project.
The text was updated successfully, but these errors were encountered: