Skip to content

Conversation

larryliu0820
Copy link
Contributor

Summary: As titled. Got too excited in #3062 and removed EXECUTORCH_BUILD_QUANTIZED. Looking at the CI job failure of build-apple-framework probably worth adding it back.

Test Plan: See that CI job pass

Reviewers:

Subscribers:

Tasks:

Tags:

Copy link

pytorch-bot bot commented Apr 18, 2024

🔗 Helpful Links

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

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

❌ 3 New Failures

As of commit dc9b3b3 with merge base 910f851 (image):

NEW FAILURES - The following jobs have failed:

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

@larryliu0820 larryliu0820 requested a review from shoumikhin April 18, 2024 01:04
@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 18, 2024
@facebook-github-bot
Copy link
Contributor

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

Summary:
As titled. Got too excited in #3062 and removed `EXECUTORCH_BUILD_QUANTIZED`. Looking at the CI job failure of `build-apple-framework` probably worth adding it back.


Test Plan: See that CI job pass

Reviewed By: shoumikhin

Differential Revision: D56281923

Pulled By: larryliu0820
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D56281923

@facebook-github-bot
Copy link
Contributor

@larryliu0820 merged this pull request in eb47c4e.

@mergennachin mergennachin mentioned this pull request Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants