Skip to content

Conversation

@mikaylagawarecki
Copy link
Contributor

@mikaylagawarecki mikaylagawarecki commented Nov 5, 2025

@pytorch-bot
Copy link

pytorch-bot bot commented Nov 5, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 54ed4b3 with merge base c724f00 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
// Put result back on stack using `from()`
// Stack slot 0 now holds the return value
stack[0] = from(result);
Copy link
Contributor

Choose a reason for hiding this comment

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

oooh should we also cherrypick the torch::stable::detail::from change + migrate these to use that? (general q for outside of this PR)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

up to you

mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
@mikaylagawarecki mikaylagawarecki marked this pull request as ready for review November 5, 2025 21:10
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 5, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 6, 2025
mikaylagawarecki added a commit that referenced this pull request Nov 6, 2025
@mikaylagawarecki
Copy link
Contributor Author

@pytorchbot merge

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Nov 6, 2025
@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

@mikaylagawarecki mikaylagawarecki added release notes: cpp release notes category topic: docs topic category and removed topic: not user facing topic category labels Nov 6, 2025
atalman pushed a commit that referenced this pull request Nov 7, 2025
… (#167323)

* [BE] Refresh documentation for stable ABI / API (#163899)

Pull Request resolved: #163899
Approved by: https://github.com/janeyx99

* Document LibTorch ABI more, add README to headeronly (#166661)

Pull Request resolved: #166661
Approved by: https://github.com/mikaylagawarecki, https://github.com/albanD

* Add guidance on how to migrate kernels to the libtorch stable ABI (#167112)

Pull Request resolved: #167112
Approved by: https://github.com/janeyx99

---------

Co-authored-by: Jane Xu <[email protected]>
@github-actions github-actions bot deleted the gh/mikaylagawarecki/366/head branch December 7, 2025 02:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk Trigger trunk jobs on your pull request Merged release notes: cpp release notes category topic: docs topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants