Skip to content

Conversation

fumchin
Copy link
Collaborator

@fumchin fumchin commented Apr 7, 2025

The sqrt operation is implemented via the pow operator, sqrt(x) = pow(x, 0.5).

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

@fumchin fumchin added 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 ciflow/trunk topic: not user facing labels Apr 7, 2025
@fumchin fumchin requested a review from digantdesai as a code owner April 7, 2025 08:13
Copy link

pytorch-bot bot commented Apr 7, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 7a4622b with merge base 91a14f1 (image):
💚 Looks good so far! There are no failures yet. 💚

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

- implement sqrt as pow(x, 0.5)
- Added tests for sqrt

Signed-off-by: Fang-Ching <[email protected]>
Change-Id: I24c8fcbbafbca8e341825aa6792bfb1eaf426194
Copy link
Collaborator

@oscarandersson8218 oscarandersson8218 left a comment

Choose a reason for hiding this comment

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

Looks good to me! CI failures should be unrelated. Just waiting for #9451 to be merged before merging this.

@zingo zingo merged commit 7048740 into pytorch:main Apr 11, 2025
167 checks passed
kirklandsign pushed a commit that referenced this pull request Apr 11, 2025
The sqrt operation is implemented via the pow operator, sqrt(x) = pow(x,
0.5).

Signed-off-by: Fang-Ching <[email protected]>
keyprocedure pushed a commit to keyprocedure/executorch that referenced this pull request Apr 21, 2025
The sqrt operation is implemented via the pow operator, sqrt(x) = pow(x,
0.5).

Signed-off-by: Fang-Ching <[email protected]>
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