-
Notifications
You must be signed in to change notification settings - Fork 473
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
Android improve prebuilt workflow and provide tutorial #3840
Conversation
kirklandsign
commented
Jun 4, 2024
- Don't depend on demo app. Instead, build JAR/JNI/AAR directly
- Update tutorial, example scripts for LLAMA demo app
Don't depend on demo app. Instead, build JAR/JNI/AAR directly
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/3840
Note: Links to docs will display an error until the docs builds have been completed. ❌ 2 New FailuresAs of commit ea38c68 with merge base f184329 ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
SHA is temporary! |
@kirklandsign has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we actually want to hard-code "0.2.1" in the main branch?
@@ -0,0 +1,37 @@ | |||
# Using Android prebuilt libraries (AAR) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add a link for this page from here?
https://docs-preview.pytorch.org/pytorch/executorch/3840/demo-apps-android.html?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So far I added a link in LLAMA demo app page, because I set up LLAMA in a way that we can use either prebuilt library or locally built library.
For ExecuTorch demo app, I can't easily modify the code to let user choose between two options. Especially when the ExecuTorch demo app has Qualcomm related stuff. I feel it's risky. Then linking the prebuilt library instruction to ExecuTorch demo app page might confuse people, because they need to modify the gradle rule a bit.
Let me know what you think.
@kirklandsign merged this pull request in e85b52a. |
I will update it for main. I use 0.2.1 so it's easier to CP to release branch |