KEMBAR78
Move complex to headeronly by janeyx99 · Pull Request #159411 · pytorch/pytorch · GitHub
Skip to content

Conversation

@janeyx99
Copy link
Contributor

@janeyx99 janeyx99 commented Jul 29, 2025

[ghstack-poisoned]
@pytorch-bot
Copy link

pytorch-bot bot commented Jul 29, 2025

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit 7930e44 with merge base d2e0258 (image):

UNSTABLE - The following job is marked as unstable, possibly due to flakiness on trunk:

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

janeyx99 added a commit that referenced this pull request Jul 29, 2025
ghstack-source-id: 5b37da8
Pull Request resolved: #159411
@janeyx99 janeyx99 marked this pull request as draft July 29, 2025 20:15
janeyx99 added a commit that referenced this pull request Jul 30, 2025
ghstack-source-id: 5e6b912
Pull Request resolved: #159411
@janeyx99 janeyx99 marked this pull request as ready for review July 30, 2025 05:58
@janeyx99 janeyx99 added ciflow/trunk Trigger trunk jobs on your pull request release notes: build release notes category labels Jul 30, 2025
@janeyx99 janeyx99 requested review from albanD and desertfire July 30, 2025 05:58
#include <torch/headeronly/util/complex.h>

// std functions
//
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are the followings not moved?

Copy link
Contributor Author

@janeyx99 janeyx99 Jul 30, 2025

Choose a reason for hiding this comment

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

There is quite a lot in here, and complex_math.h is not header only, so I just moved the APIs I cared about.

Copy link
Collaborator

@albanD albanD left a comment

Choose a reason for hiding this comment

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

SGTM

janeyx99 added a commit that referenced this pull request Jul 31, 2025
ghstack-source-id: 5d92c0d
Pull Request resolved: #159411
janeyx99 added a commit that referenced this pull request Jul 31, 2025
ghstack-source-id: 53d3938
Pull Request resolved: #159411
@janeyx99
Copy link
Contributor Author

@pytorchbot merge

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

yangw-dev pushed a commit that referenced this pull request Aug 1, 2025
Pull Request resolved: #159411
Approved by: https://github.com/albanD
ghstack dependencies: #159415
pytorchmergebot pushed a commit that referenced this pull request Aug 1, 2025
Pull Request resolved: #159416
Approved by: https://github.com/albanD
ghstack dependencies: #159415, #159411
@github-actions github-actions bot deleted the gh/janeyx99/287/head branch August 31, 2025 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/inductor ciflow/trunk Trigger trunk jobs on your pull request Merged release notes: build release notes category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants