Skip to content

Cherry pick #3006 #3190

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

Merged
merged 3 commits into from
Apr 22, 2024
Merged

Cherry pick #3006 #3190

merged 3 commits into from
Apr 22, 2024

Conversation

huydhn
Copy link
Contributor

@huydhn huydhn commented Apr 21, 2024

This is to fix the broken iOS frameworks build job on release/0.2 branch as seen in https://hud.pytorch.org/hud/pytorch/executorch/release%2F0.2/1?per_page=50&name_filter=apple and #3134

#2996 has already been cherry picked by #3186

Summary:
As titled. `build_apple_frameworks.sh` is copying all the exported headers out and in #2934 `//executorch/schema:program` is being moved to `exported_deps` and causing `build_apple_frameworks.sh` to not able to copy generated headers `program_generated.h` and `scalar_type_generated.h`.

This PR fixes it by moving it back to `deps`.

Pull Request resolved: #2996

Reviewed By: kirklandsign

Differential Revision: D56028952

Pulled By: larryliu0820

fbshipit-source-id: 2cd4999154877b0ac7b49cd1f54d518cba34b2f2
Summary:
* Apple / build-frameworks-ios / macos-job

We removed libcustom_ops_lib.a in #2916 so need to remove it from `build_apple_frameworks.sh`.

* Lint / lintrunner / linux-job

Remove extra line in backends/qualcomm/quantizer/utils.py

* pull / unittest / macos (buck2) / macos-job

Fix it by using `executorch_no_prim_ops` instead of `executorch` in MPS and CoreML.

Pull Request resolved: #3006

Reviewed By: lucylq

Differential Revision: D56048430

Pulled By: larryliu0820

fbshipit-source-id: 9dcb476eea446ea3aba566d595167c691fb00eec
@huydhn huydhn requested review from larryliu0820 and cccclai April 21, 2024 00:50
Copy link

pytorch-bot bot commented Apr 21, 2024

🔗 Helpful Links

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

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

❌ 2 New Failures

As of commit c75f7c7 with merge base d3326a2 (image):

NEW FAILURES - The following jobs have failed:

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 Apr 21, 2024
@huydhn huydhn requested a review from guangy10 April 21, 2024 00:50
@cccclai
Copy link
Contributor

cccclai commented Apr 21, 2024

thank you!

@cccclai
Copy link
Contributor

cccclai commented Apr 22, 2024

#2996 is merged instead

@cccclai cccclai closed this Apr 22, 2024
@cccclai cccclai reopened this Apr 22, 2024
@huydhn huydhn changed the title Cherry pick #2996 and #3006 Cherry pick #3006 Apr 22, 2024
@huydhn huydhn merged commit dcd5c44 into release/0.2 Apr 22, 2024
@mergennachin mergennachin mentioned this pull request Apr 25, 2024
@huydhn huydhn deleted the cherry-pick-2996-and-3006 branch August 26, 2024 21:55
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants