Skip to content

Conversation

Sebastian-Larsson
Copy link
Collaborator

@Sebastian-Larsson Sebastian-Larsson commented Apr 11, 2025

Asserts are converted to proper raises to ensure graph integrity.

Improve error messages and add additional check that both input and output are of data type fp32.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

Asserts are converted to proper raises to ensure graph integrity.

Improve error messages and add additional check that both input and
output are of data type fp32.

Signed-off-by: Sebastian Larsson <sebastian.larsson@arm.com>
Change-Id: Ied5b0b13f061527d0b01cb864f6716e646bb8db3
Copy link

pytorch-bot bot commented Apr 11, 2025

🔗 Helpful Links

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

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

⏳ No Failures, 2 Pending

As of commit 5b244e8 with merge base a073668 (image):
💚 Looks good so far! There are no failures yet. 💚

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 Apr 11, 2025
@Sebastian-Larsson Sebastian-Larsson added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk topic: not user facing CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. and removed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Apr 11, 2025
@zingo zingo merged commit c38e71f into pytorch:main Apr 11, 2025
167 of 170 checks passed
kirklandsign pushed a commit that referenced this pull request Apr 11, 2025
Asserts are converted to proper raises to ensure graph integrity.

Improve error messages and add additional check that both input and
output are of data type fp32.

Signed-off-by: Sebastian Larsson <sebastian.larsson@arm.com>
keyprocedure pushed a commit to keyprocedure/executorch that referenced this pull request Apr 21, 2025
Asserts are converted to proper raises to ensure graph integrity.

Improve error messages and add additional check that both input and
output are of data type fp32.

Signed-off-by: Sebastian Larsson <sebastian.larsson@arm.com>
@Sebastian-Larsson Sebastian-Larsson deleted the assert_rsqrt branch May 15, 2025 12:40
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 topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants