Skip to content

[OpenReg] Add Develop Notes for Integrating New Backend into PyTorch#158644

Closed
fffrog wants to merge 18 commits intogh/fffrog/124/basefrom
gh/fffrog/124/head
Closed

[OpenReg] Add Develop Notes for Integrating New Backend into PyTorch#158644
fffrog wants to merge 18 commits intogh/fffrog/124/basefrom
gh/fffrog/124/head

Conversation

@fffrog
Copy link
Collaborator

@fffrog fffrog commented Jul 18, 2025

Stack from ghstack (oldest at bottom):

To facilitate the integration of the new backend, we plan to publish a new development note that details all the key components,hoping to speed up the development of other accelerators.

This PR is the beginning of this note, and involve the part of registration of operators and we will gradually improve it and keep in sync with OpenReg's code.

[ghstack-poisoned]
@fffrog fffrog requested review from albanD and jeffdaily as code owners July 18, 2025 12:22
@pytorch-bot
Copy link

pytorch-bot bot commented Jul 18, 2025

🔗 Helpful Links

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

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

❌ 1 New Failure, 1 Unrelated Failure

As of commit 0d85743 with merge base 9a12bab (image):

NEW FAILURE - The following job has failed:

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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

@pytorch-bot pytorch-bot bot added the release notes: releng release notes category label Jul 18, 2025
fffrog added a commit that referenced this pull request Jul 18, 2025
As the title stated.

To facilitate the integration of the new backend, we plan to publish a new
development note that details all the key components, hoping to speed up
the development of other accelerators.

This PR is the beginning of this note, which we will gradually improve and
keep in sync with OpenReg's code.


ghstack-source-id: 6b8c629
Pull-Request-resolved: #158644
[ghstack-poisoned]
fffrog added a commit that referenced this pull request Jul 18, 2025
As the title stated.

To facilitate the integration of the new backend, we plan to publish a new
development note that details all the key components, hoping to speed up
the development of other accelerators.

This PR is the beginning of this note, which we will gradually improve and
keep in sync with OpenReg's code.

ghstack-source-id: 7986da4
Pull-Request-resolved: #158644
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.

Thanks for starting this!
I'm sure there is going to be a LOT of things to add there. But very exciting to see this first draft!

[ghstack-poisoned]
@fffrog fffrog marked this pull request as draft July 21, 2025 08:05
fffrog added a commit that referenced this pull request Jul 21, 2025
As the title stated.

To facilitate the integration of the new backend, we plan to publish a new
development note that details all the key components, hoping to speed up
the development of other accelerators.

This PR is the beginning of this note, which we will gradually improve and
keep in sync with OpenReg's code.

ghstack-source-id: 9c3aee3
Pull-Request-resolved: #158644
[ghstack-poisoned]
fffrog added a commit that referenced this pull request Jul 23, 2025
As the title stated.

To facilitate the integration of the new backend, we plan to publish a new
development note that details all the key components, hoping to speed up
the development of other accelerators.

This PR is the beginning of this note, which we will gradually improve and
keep in sync with OpenReg's code.

ghstack-source-id: 9ea01fc
Pull-Request-resolved: #158644
[ghstack-poisoned]
fffrog added a commit that referenced this pull request Jul 25, 2025
As the title stated.

To facilitate the integration of the new backend, we plan to publish a new
development note that details all the key components, hoping to speed up
the development of other accelerators.

This PR is the beginning of this note, which we will gradually improve and
keep in sync with OpenReg's code.

ghstack-source-id: 2e10e33
Pull-Request-resolved: #158644
[ghstack-poisoned]
@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Aug 1, 2025
fffrog added a commit that referenced this pull request Aug 1, 2025
As the title stated.

To facilitate the integration of the new backend, we plan to publish a new
development note that details all the key components, hoping to speed up
the development of other accelerators.

This PR is the beginning of this note, which we will gradually improve and
keep in sync with OpenReg's code.

ghstack-source-id: 275a5c1
Pull-Request-resolved: #158644
@fffrog fffrog marked this pull request as ready for review August 1, 2025 09:14
@fffrog fffrog changed the title [OpenReg] Add Develop Notes for Integrating New Backend into PyTorch [OpenReg] Add Develop Notes for Integrating New Backend into PyTorch(Operator Aspect) Aug 1, 2025
[ghstack-poisoned]
[ghstack-poisoned]
can-gaa-hou pushed a commit to can-gaa-hou/pytorch that referenced this pull request Aug 14, 2025
To facilitate the integration of the new backend, we plan to publish a new development note that details all the key components,hoping to speed up the development of other accelerators.

This PR is the beginning of this note, and involve the part of registration of operators and we will gradually improve it and keep in sync with OpenReg's code.

ghstack-source-id: 29c7776
Pull-Request-resolved: pytorch#158644
fffrog added 2 commits August 19, 2025 16:45
[ghstack-poisoned]
[ghstack-poisoned]
can-gaa-hou pushed a commit to can-gaa-hou/pytorch that referenced this pull request Aug 22, 2025
To facilitate the integration of the new backend, we plan to publish a new development note that details all the key components,hoping to speed up the development of other accelerators.

This PR is the beginning of this note, and involve the part of registration of operators and we will gradually improve it and keep in sync with OpenReg's code.

ghstack-source-id: 915366b
Pull-Request-resolved: pytorch#158644
[ghstack-poisoned]
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.

Looks great!
Thanks for the time and effort in compiling all of this.

As this document grows we might want to have a shorter intro at the top about the big axis: c++ scaffolding, operators, testing, python module.
Also mentioning for the future that in the latest branch, we started a user guide section in the doc that might be a good place if we want to have multiple pages nicely tied together: https://docs.pytorch.org/docs/main/user_guide/index.html

All the comments are pretty minor, please add the appropriate external link and then you can merge!

[ghstack-poisoned]
[ghstack-poisoned]
@fffrog
Copy link
Collaborator Author

fffrog commented Aug 27, 2025

As this document grows we might want to have a shorter intro at the top about the big axis: c++ scaffolding, operators, testing, python module.

Got it, I will add it in the follow-up PRs.

Also mentioning for the future that in the latest branch, we started a user guide section in the doc that might be a good place if we want to have multiple pages nicely tied together: https://docs.pytorch.org/docs/main/user_guide/index.html

Thank you very much for these very helpful suggestions.
As the documentation expands, it will contain more and more information. If we can split it into several separate files, it will not only be easier for users to read but also for developers to maintain.

So, is it possible to create a separate section within the User Guide (on the same level as the Developer Notes) and then split the documentation into sections with granularity like Overall, Operators, Runtime and so on? This might provide more clarity.

@fffrog
Copy link
Collaborator Author

fffrog commented Aug 27, 2025

@pytorchbot merge

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

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: 1 jobs have failed, first few of them are: trunk / linux-jammy-rocm-py3.10 / test (default, 2, 2, linux.rocm.gpu.gfx942.1)

Details for Dev Infra team Raised by workflow job

@fffrog
Copy link
Collaborator Author

fffrog commented Aug 27, 2025

@pytorchbot merge -i

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged while ignoring the following 2 checks: trunk / macos-py3-arm64 / test (default, 2, 3, macos-m1-stable), trunk / linux-jammy-rocm-py3.10 / test (default, 2, 2, linux.rocm.gpu.gfx942.1)

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

pytorchmergebot pushed a commit that referenced this pull request Aug 30, 2025
markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 2025
…ytorch#158644)

To facilitate the integration of the new backend, we plan to publish a new development note that details all the key components,hoping to speed up the development of other accelerators.

This PR is the beginning of this note, and involve the part of registration of operators and we will gradually improve it and keep in sync with OpenReg's code.
Pull Request resolved: pytorch#158644
Approved by: https://github.com/albanD
markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 2025
mansiag05 pushed a commit to mansiag05/pytorch that referenced this pull request Sep 22, 2025
@github-actions github-actions bot deleted the gh/fffrog/124/head branch September 27, 2025 02:07
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 open source release notes: releng release notes category topic: not user facing topic category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants