-
Notifications
You must be signed in to change notification settings - Fork 6.8k
Make amalgamation part be suitable for iOS #11184
Conversation
@KellenSunderland anyone you know is familiar with amalgamation and could help review? |
@Aozorany thanks for your contribution!
|
@Aozorany bounce... |
Looping around on this for @eric-haibin-lin. Sorry not aware of anyone with more than a passing knowledge of amalgamation. |
sorry for the delay, really got lots of things to do, i'll finish the end of the week. |
seems something was changed in master branch since my PR, so i'll working on it to fix. i cloned master branch and want to compile the predict part on iOS, i followed the instructions of the document installed openblas and all 3rd party libs, type
looks like it missed some symbols in file |
i think i'll get more time to look into it, things have changed since my last investigation, my method runs all right in version 1.2.1, but not working in the current head version. |
@mxnet-label-bot[pr-work-in-progress] |
Continuing with pr-work-in-progress label unless there is an update |
Hi @Aozorany, Thanks for working on fixing this. Any update on this PR? |
ping @Aozorany could you fix the CI lint error in |
@Aozorany ping! could you please address the issues above so we can review the PR on our end and merge it? :) |
@kalyc this PR is working fine for version 1.2.0 and 1.2.1, but getting compile errors on current master branch, i am now working on it to fix the issues. |
@Aozorany let us know if you need any help on compile errors! |
@stu1130 could this amalgamation part compile successfully on android? |
@Aozorany we were refactoring our CI system for the past two weeks and it's done! Could you rebase it? |
@mxnet-label-bot add[pr-awaiting-response] |
@Aozorany - Requesting you to look at failed CI builds. |
@Aozorany Did you get a chance to look into failing CI builds? |
@Aozorany bounce again for the update! Thanks |
@Aozorany - How can we help here? What are the issues currently you are facing master? |
@mxnet-label-bot remove[pr-work-in-progress] |
@nswamy @sandeep-krishnamurthy @anirudh2290 - Please consider closing this PR since there is no follow up from the author since December. |
@Aozorany Could you please provide an update on the PR? |
@Aozorany This PR seems to be inactive for a long time. If there's any help you need to take this PR forward please feel free to reach out on this PR. |
Description
Fixed some errors when make amalgamation for iOS use, now this part is OK to use.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.
Changes