Skip to content

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
@pytorch-bot
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