Skip to content

Implement _fft_c2r core ATen op #10208

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 1 commit into from
Apr 19, 2025

Conversation

pssrawat
Copy link
Contributor

Summary: Add ff2_c2r

Differential Revision: D73006888

Copy link

pytorch-bot bot commented Apr 15, 2025

🔗 Helpful Links

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

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:

❌ 5 New Failures, 1 Cancelled Job, 52 Pending, 2 Unrelated Failures

As of commit 1a3f991 with merge base 4401fab (image):

NEW FAILURES - The following jobs have failed:

CANCELLED JOB - The following job was cancelled. Please retry:

FLAKY - The following jobs failed but were likely due to flakiness present on trunk:

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 15, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73006888

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73006888

pssrawat added a commit to pssrawat/executorch that referenced this pull request Apr 15, 2025
Summary:
Pull Request resolved: pytorch#10208

Add ff2_c2r

Differential Revision: D73006888

namespace torch::executor::native {

// TODO: contents of this anonymous namespace are copy/pasted from
Copy link
Contributor

Choose a reason for hiding this comment

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

A lot of this looks copy/pasted from op_fft_r2c.cpp . Please don't do that; instead, create a header that both op_fft_r2c.cpp and this file can include.

@pssrawat pssrawat requested a review from swolchok April 18, 2025 16:29
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73006888

pssrawat added a commit to pssrawat/executorch that referenced this pull request Apr 18, 2025
Summary:
Pull Request resolved: pytorch#10208

Add ff2_c2r

Differential Revision: D73006888
Copy link
Contributor

@swolchok swolchok left a comment

Choose a reason for hiding this comment

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

seems OK now

Summary:
Pull Request resolved: pytorch#10208

Add ff2_c2r

Reviewed By: swolchok

Differential Revision: D73006888
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D73006888

@facebook-github-bot facebook-github-bot merged commit ae8e5be into pytorch:main Apr 19, 2025
67 of 82 checks passed
keyprocedure pushed a commit to keyprocedure/executorch that referenced this pull request Apr 21, 2025
Differential Revision: D73006888

Pull Request resolved: pytorch#10208
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants