Skip to content
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

Create intermediate directories in ETCoreMLModelManager #9562

Merged
merged 2 commits into from
Mar 25, 2025

Conversation

metascroy
Copy link
Contributor

@metascroy metascroy commented Mar 24, 2025

ETCoreML crashes in FB app. During debugging, we traced issue down to this.

cc @kimishpatel @YifanShenSZ @cymbalrush

Copy link

pytorch-bot bot commented Mar 24, 2025

🔗 Helpful Links

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

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

⏳ 62 Pending, 2 Unrelated Failures

As of commit 34ea2dd with merge base 644b7dd (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

BROKEN TRUNK - The following job failed but was present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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 24, 2025
@metascroy metascroy added ciflow/trunk module: coreml Issues related to Apple's Core ML delegation and code under backends/apple/coreml/ labels Mar 24, 2025
@metascroy
Copy link
Contributor Author

@cymbalrush can you have a look?

@metascroy
Copy link
Contributor Author

@pytorchbot label "topic: not user facing"

@facebook-github-bot
Copy link
Contributor

@metascroy 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.

Approve on behalf of @cymbalrush

@facebook-github-bot
Copy link
Contributor

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

@metascroy metascroy merged commit c20e8ab into main Mar 25, 2025
166 of 169 checks passed
@metascroy metascroy deleted the fix-coreml-fb-app branch March 25, 2025 18:39
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. module: coreml Issues related to Apple's Core ML delegation and code under backends/apple/coreml/ topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants