[OpenReg] Add Develop Notes for Integrating New Backend into PyTorch#158644
[OpenReg] Add Develop Notes for Integrating New Backend into PyTorch#158644fffrog wants to merge 18 commits intogh/fffrog/124/basefrom
Conversation
🔗 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 FailureAs of commit 0d85743 with merge base 9a12bab ( 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. |
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
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
albanD
left a comment
There was a problem hiding this comment.
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!
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
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
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
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
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
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
albanD
left a comment
There was a problem hiding this comment.
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!
Got it, I will add it in the follow-up PRs.
Thank you very much for these very helpful suggestions. 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 |
|
@pytorchbot merge |
Merge startedYour 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 |
Merge failedReason: 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 teamRaised by workflow job |
|
@pytorchbot merge -i |
Merge startedYour 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 |
…61603) As the title stated. Related Infos: #158644 (comment) Pull Request resolved: #161603 Approved by: https://github.com/albanD
…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
…torch#161603) As the title stated. Related Infos: pytorch#158644 (comment) Pull Request resolved: pytorch#161603 Approved by: https://github.com/albanD
…torch#161603) As the title stated. Related Infos: pytorch#158644 (comment) Pull Request resolved: pytorch#161603 Approved by: https://github.com/albanD
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.