Skip to content

Update Arm delegate tutorial for MobileNetV2 usages #3914

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 2 commits into from
Closed

Update Arm delegate tutorial for MobileNetV2 usages #3914

wants to merge 2 commits into from

Conversation

Jerry-Ge
Copy link
Contributor

Add the related setup documentations for running
the Delegated Quantized MobileNetV2 example

Change-Id: Ied8298938fcb37cfa95bd044439c305380f0c1eb

@Jerry-Ge
Update Arm delegate tutorial for MobileNetV2 usages
b8dc1f6 
Add the related setup documentations for running
the Delegated Quantized MobileNetV2 example

Signed-off-by: Jerry Ge <[email protected]>
Change-Id: Ied8298938fcb37cfa95bd044439c305380f0c1eb
@pytorch-bot PyTorch Bot
Copy link

pytorch-bot bot commented Jun 10, 2024 •
edited
Loading

🔗 Helpful Links

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

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

✅ No Failures

As of commit c925191 with merge base 0bb5492 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@Jerry-Ge Jerry-Ge requested a review from robell June 10, 2024 17:05
@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 Jun 10, 2024
digantdesai
digantdesai approved these changes Jun 10, 2024
View reviewed changes
docs/source/executorch-arm-delegate-tutorial.md

At the end of this, we should have two different `.pte` files. First one with the [SoftmaxModule](#softmaxmodule), without any backend delegates. And the second one with the [AddModule](#addmodule), and with Arm Ethos-U backend delegate enabled. Now let's try to run these `.pte` files on a Corstone-300 platform in a bare-metal environment.
### Delegated Quantized Workflow
Before generating the `.pte` file for delegated quantized networks like MobileNetV2, we need to build the `quantized_ops_aot_lib`
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Is this covered by some other docs? If not does it make sense to create another page and link it from here? Rationale is if other use cases need to do this as well.

Copy link
Contributor Author

@Jerry-Ge Jerry-Ge Jun 10, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this covered by some other docs?

No from my understanding. That makes sense. I will refactor this part once we got more use cases. I will leave this as this right now.

@facebook-github-bot
Copy link
Contributor

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

@digantdesai digantdesai added the partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm label Jun 10, 2024
@digantdesai
Copy link
Contributor

Also you might want to rebase on top of the #3847 to make sure the CI is clean

@Jerry-Ge
Copy link
Contributor Author

Also you might want to rebase on top of the #3847 to make sure the CI is clean

just rebased! thanks for the note.

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

@digantdesai merged this pull request in a7a9bac.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@digantdesai digantdesai digantdesai approved these changes

@robell robell Awaiting requested review from robell

@mergennachin mergennachin Awaiting requested review from mergennachin mergennachin is a code owner

Assignees
No one assigned
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Jerry-Ge @facebook-github-bot @digantdesai