-
Notifications
You must be signed in to change notification settings - Fork 607
iOS demo app. #557
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
iOS demo app. #557
Conversation
✅ Deploy Preview for resplendent-gnome-14e531 canceled.
|
This pull request was exported from Phabricator. Differential Revision: D49800418 |
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.
for model files bundled here, I am gonna suggest to have a python script that generates this files and link them as steps in the tutorial:
e.g.
- python generate_mv2.py --delegate...
- copy the pte file to
- ..
isa = PBXFrameworksBuildPhase; | ||
buildActionMask = 2147483647; | ||
files = ( | ||
03C818472AC7A1190084CC29 /* libImageClassification.a in Frameworks */, |
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.
what are the ids? and how are they connect to libs
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.
These are not related, they are part of the xcode project, which is currently hardcoded, but we'll generate it eventually and get rid of this file.
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.
sending it back for 2 things
- if we can make model generation as part of the script that should be invoked, that would be good.
- documentation as to how to enable your own delegate. this will help apple
...ples/ios_demo_apps/ExecuTorchDemo/ExecuTorchDemo/Sources/MobileNet/MobileNetClassifier.swift
Show resolved
Hide resolved
...ples/ios_demo_apps/ExecuTorchDemo/ExecuTorchDemo/Sources/MobileNet/MobileNetClassifier.swift
Show resolved
Hide resolved
...ples/ios_demo_apps/ExecuTorchDemo/ExecuTorchDemo/Sources/MobileNet/MobileNetClassifier.swift
Show resolved
Hide resolved
std::vector<torch::executor::Span<uint8_t>> plannedSpans_; | ||
std::unique_ptr<torch::executor::HierarchicalAllocator> plannedMemory_; | ||
std::unique_ptr<torch::executor::MemoryManager> memoryManager_; | ||
std::unique_ptr<torch::executor::Method> forward_; |
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.
maybe call it method_
for consistency
examples/ios_demo_apps/ExecuTorchDemo/ExecuTorchDemo/Sources/MobileNet/Module.cpp
Show resolved
Hide resolved
...ples/ios_demo_apps/ExecuTorchDemo/ExecuTorchDemo/Sources/MobileNet/MobileNetClassifier.swift
Show resolved
Hide resolved
examples/ios_demo_apps/ExecuTorchDemo/ExecuTorchDemo/Sources/MobileNet/MobileNetClassifier.mm
Show resolved
Hide resolved
examples/ios_demo_apps/ExecuTorchDemo/ExecuTorchDemo/Sources/MobileNet/Module.cpp
Show resolved
Hide resolved
This pull request was exported from Phabricator. Differential Revision: D49800418 |
1 similar comment
This pull request was exported from Phabricator. Differential Revision: D49800418 |
for ios toolchain for cmake, I think we should standardize it. Apple's PR also has sometihng like that but it is part of their PR |
This pull request was exported from Phabricator. Differential Revision: D49800418 |
2 similar comments
This pull request was exported from Phabricator. Differential Revision: D49800418 |
This pull request was exported from Phabricator. Differential Revision: D49800418 |
They should be able to use ours, as shown in PR summary |
This pull request was exported from Phabricator. Differential Revision: D49800418 |
4 similar comments
This pull request was exported from Phabricator. Differential Revision: D49800418 |
This pull request was exported from Phabricator. Differential Revision: D49800418 |
This pull request was exported from Phabricator. Differential Revision: D49800418 |
This pull request was exported from Phabricator. Differential Revision: D49800418 |
This pull request was exported from Phabricator. Differential Revision: D49800418 |
Summary: Pull Request resolved: #557 Reviewed By: cccclai Differential Revision: D49800418 fbshipit-source-id: c4233781166453bca131c5fe0c2d76ad60f20448
This pull request was exported from Phabricator. Differential Revision: D49800418 |
This pull request has been merged in f9e23ea. |
Summary:
Pre-flight:
pip
pointing to itPost-flight:
Differential Revision: D49800418