-
Notifications
You must be signed in to change notification settings - Fork 722
Handle model state in benchmark CI jobs #7432
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
base: main
Are you sure you want to change the base?
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/7432
Note: Links to docs will display an error until the docs builds have been completed. ❌ 6 New FailuresAs of commit e195698 with merge base a727b55 ( NEW FAILURES - The following jobs have failed:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
|
@guangy10 the apple-perf/build-benchmark-app job fails with "xcodebuild: error: Unable to find a device matching the provided destination specifier:2025-01-02T18:01:02.3778750Z { platform:iOS }" Do you know what could cause this? |
guangy10
left a comment
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.
Thanks @metascroy for improvement!
| secrets: inherit | ||
| with: | ||
| runner: macos-latest-xlarge | ||
| runner: macos-15-xlarge |
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.
This runner is configured with Xcode 16.0 right? Does it work for all other models that running on iOS 17?
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.
Most runners should have xcode 16, but you have to set the version line I do on line 333.
@guangy10 do you understanding the failing build-benchmark-app errror in CI: apple-perf/build-benchmark-app job fails with "xcodebuild: error: Unable to find a device matching the provided destination specifier:2025-01-02T18:01:02.3778750Z { platform:iOS }" |
Can you share the job link? Can't find this error in the failed job. However, I'm seeing the missing stdlib errors again in pretty much all app builds, for example, https://github.com/pytorch/executorch/actions/runs/12586454085/job/35080314575?pr=7432 Maybe @shoumikhin can help with it? |
The required |
|
Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as |
|
Looks like this PR hasn't been updated in a while so we're going to go ahead and mark this as |
Previously when running a pte file that has model state, our CI would fail with an error like:
This is because if you build the xcode frameworks with xcode 15, the ExecuTorch CoreML backend is built to not handle MLState variables:
executorch/backends/apple/coreml/runtime/delegate/MLModel_Prewarm.h
Line 11 in fc42a4e
Note the prebuilt frameworks cannot handle MLState (https://github.com/pytorch/executorch/tree/main/extension/benchmark/apple/Benchmark#set-up-the-frameworks).
This PR: