Skip to content

Conversation

kirklandsign
Copy link
Contributor

@kirklandsign kirklandsign commented Mar 13, 2025

Summary

We have

which is E2E for building the test. We don't need it when we build the AAR. This reduces the time for user to build AAR.

For benchmark, before, it took 30 minutes to build; now it takes 20 minutes.

Test plan

CI

@pytorch-bot
Copy link

pytorch-bot bot commented Mar 13, 2025

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/9249

Note: Links to docs will display an error until the docs builds have been completed.

⏳ No Failures, 4 Pending

As of commit ad252fa with merge base ce612b8 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Mar 13, 2025
@kirklandsign kirklandsign changed the title Split android instrumentation Split android instrumentation from build script Mar 13, 2025
@kirklandsign kirklandsign temporarily deployed to upload-benchmark-results March 13, 2025 23:52 — with GitHub Actions Inactive
@facebook-github-bot
Copy link
Contributor

@kirklandsign has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Copy link
Contributor

@cccclai cccclai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like big improvement on the build time

@facebook-github-bot facebook-github-bot merged commit 9a0c2db into main Mar 14, 2025
84 of 89 checks passed
@facebook-github-bot facebook-github-bot deleted the android-skip-demo-app-2 branch March 14, 2025 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. topic: not user facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants