amino-os / Amino.Run-Demos

Amino.Run - Demo Applications
Apache License 2.0
5 stars 6 forks source link

Update the GoGameAndroid app to make it work with the latest changes in the amino code #42

Open Vishwa4jeet opened 5 years ago

Vishwa4jeet commented 5 years ago
  1. Try and run the app.
  2. Update the app with the latest changes in the amino code and ensure successful build.
Vishwa4jeet commented 5 years ago

Cloned the app from the GoGameAndroid branch, and tried running it with its current state. Following issue were obeserved:

Finally we are able to make Kernel Server up and Pachi Micro Service deployed. But at some point it is trying to make Pachi engine up as a process. Build process failed over there with error "not able to find Pachi binary". @AmitRoushan and I have tried to debug and we located Pachi binary but location of binary is not as per path of build. We are not sure why this error and where to put binary.

Vishwa4jeet commented 5 years ago

@quinton-hoole-2 @sungwook-moon We are currently stuck at this point. So, any input would be valuable in driving this forward.

sungwook-moon commented 5 years ago

I discussed with Hong and we perhaps need to discuss further in our next sync-up. Basically, binary for Android should be under resources: https://github.com/Huawei-PaaS/DCAP-Sapphire-Examples/tree/GoGameAndroid/app/src/main/res

Pachi engine is C++ and the current source code uses it via Linux pipeline to execute the binary of the Pachi engine. This is because the original Pachi engine was coded in C++ and use Caffe platform which involves some challenges at making work with Amino as it is (e.g., use direct JNI like License plate)

You will need to compile and put the binary in appropriate location on Android device to make it run (local option in app). Hong mentioned this will work on kernel server running on Android but won't work on the other servers such as cloud instance or Ubuntu servers (cloud option in app). To make it run on cloud or servers, you will need to compile them and adjust a bit to make it work because C++ is platform dependent and some dependencies need to be resolved differently.