Remove the requirement to install requirements for CoreML
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 5k
- Forks
- 1.2k
- Avg merge
- 2d 10h
- Merged PRs (30d)
- 581
Description
After https://github.com/pytorch/executorch/pull/9483/, we now support CoreML out of the box for ET. However, backends/apple/coreml/scripts/install_requirements.sh still exists only to support building coreml_executor_runner. Check to see if we can eliminate this installation script overall.
cc @larryliu0820 @lucylq
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with backends/apple/coreml/scripts/install_requirements.sh and the coreml_executor_runner.xcodeproj under examples/apple/coreml/executor_runner. Review the changes in pull request 9483 and check how the runner build uses the script. Done means confirming whether the script can be removed while CoreML support and the runner build continue to work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- shell
- Domain
- build-system
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100