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

Build amalgamation using a docker #8729

Closed
OferRosenberg opened this issue Nov 20, 2017 · 11 comments
Closed

Build amalgamation using a docker #8729

OferRosenberg opened this issue Nov 20, 2017 · 11 comments

Comments

@OferRosenberg
Copy link

Is there a way to build amalgamation using the android docker-multiarch ?
Alternatively, is there a parameter or configuration that can be passed to the docker-multiarch to create an android library which is similar to the one created by amalgamation (predict only, smaller, etc.)

Thanks,
Ofer.

@larroy
Copy link
Contributor

larroy commented Nov 29, 2017

Hi @OferRosenberg Right now we are cross-compiling the full MXNet. So the answer is no. We are also considering removing amalgamation.
related: #8707

@larroy
Copy link
Contributor

larroy commented Nov 29, 2017

Why do you say "predict only"?

@edmBernard
Copy link

amalgamation don't have training API, so it only able to do prediction.

@OferRosenberg
Copy link
Author

And to my understanding, as it doesn't have the training parts, it should be smaller than the full fledge MxNet. I think there's justification for keeping a predict-only, size optimized, build (either by amalgamation or other)

@b0noI
Copy link
Contributor

b0noI commented Mar 10, 2018

proposed labels: "Question"

@wykvictor
Copy link

@larroy What's the progress on 'cross-compiling the full MXNet'? For now, what's the best approach to deploy MXnet on Android? Thanks!

@larroy
Copy link
Contributor

larroy commented Apr 4, 2018

Hi, you can use the new scripts in the ci folder: ci/build.py right now it should build for android but the build is untested.

@wykvictor
Copy link

@larroy Thanks for the advice! I build android_arm64 successfully using ci/build.py(based on commit 5a480f0 in master) .
But several changes need to be made in the process of cross compilation.
The main issue is that I need to turn off pthread/lapack in CMakeLists.txt.
Suggest to add a variable like 'Android' in it to turn off these flags automatically.

@larroy
Copy link
Contributor

larroy commented Apr 25, 2018

@wykvictor thanks for the feedback, we will likely improve this in the short future. I also think we need some kind of platform detection in CMake. I started doing that but haven't been able to finish it so far.

@ThomasDelteil
Copy link
Contributor

@OferRosenberg please consider closing if your question has been answered. Summary of the conversation:

@piyushghai
Copy link
Contributor

@sandeep-krishnamurthy Requesting this issue to be closed.
@OferRosenberg Please feel free to re-open if closed in error. :)

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

No branches or pull requests

8 participants