Skip to content

TOSA specification in Arm partitioner #6851

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

Merged
merged 2 commits into from
Nov 15, 2024

Conversation

per
Copy link
Collaborator

@per per commented Nov 14, 2024

Adds information on which TOSA specification is being used for the partitioner.

per added 2 commits November 14, 2024 11:04
The TOSA specification is added as an argument to TOSASupportedOperators to
be able to make decisions on which operators to support.

Signed-off-by: Per Åstrand <[email protected]>
Change-Id: I5ba3b9dd5f5dca54c3c6d8db29ebc9c52d5cc0f7
…ation

Change-Id: Ib7eb502861948c2f8dee995f28cbb7f2baa00afb
Copy link

pytorch-bot bot commented Nov 14, 2024

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

❌ 1 New Failure

As of commit 9582186 with merge base ecdc007 (image):

NEW FAILURE - The following job has failed:

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 Nov 14, 2024
@per per requested review from digantdesai and freddan80 November 14, 2024 12:59
@per per added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk labels Nov 14, 2024
@per
Copy link
Collaborator Author

per commented Nov 14, 2024

MacOS failure since commit 2f6d64f, not related to this change.

@freddan80 freddan80 merged commit 041b7d6 into pytorch:main Nov 15, 2024
97 of 98 checks passed
@per per deleted the tosa_specification_partitioner branch November 15, 2024 11:42

# container for all SupportedTosaOperatorCheck classes
_tosa_spec_dicts: dict[TosaSpecification, dict[str, SupportedTOSAOperatorCheck]] = {
TosaSpecification.create_from_string("TOSA-0.80.0+BI"): {},
Copy link
Contributor

Choose a reason for hiding this comment

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

Nit: Add a URL as a comment on how to find the exact version spec

lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 16, 2024
Summary: Adding dependencies for pytorch#6851

Reviewed By: larryliu0820, kirklandsign

Differential Revision: D66039382
@lucylq lucylq mentioned this pull request Nov 16, 2024
lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 16, 2024
Summary:

Adding dependencies for pytorch#6851

Reviewed By: larryliu0820, kirklandsign

Differential Revision: D66039382
lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 16, 2024
Summary:

Adding dependencies for pytorch#6851

Reviewed By: larryliu0820, kirklandsign

Differential Revision: D66039382
lucylq added a commit to lucylq/executorch-1 that referenced this pull request Nov 16, 2024
Summary:

Adding dependencies for pytorch#6851

Reviewed By: larryliu0820, kirklandsign, dvorjackz

Differential Revision: D66039382
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. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants