Update XNNPACK doc structure and add template#14873
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/14873
Note: Links to docs will display an error until the docs builds have been completed. ❗ 1 Active SEVsThere are 1 currently active SEVs. If your PR is affected, please view them below: ❌ 6 New Failures, 1 Cancelled Job, 13 PendingAs of commit ed22ca6 with merge base 7d4db45 ( NEW FAILURES - The following jobs have failed:
CANCELLED JOB - The following job was cancelled. Please retry:
This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This PR needs a
|
|
This is a pretty early draft, but I'd be interested in any thoughts. |
15169b1 to
bd30b09
Compare
bd30b09 to
42a6b71
Compare
| | Backend | Platform(s) | Hardware Type | Typical Use Case | | ||
| |-----------------------------------------------|---------------------|---------------|---------------------------------| | ||
| | [XNNPACK](backends/xnnpack/xnnpack-overview) | All | CPU | General-purpose, fallback | | ||
| | [Core ML](backends-coreml) | iOS, macOS | NPU/GPU | Apple devices, high performance | |
digantdesai
left a comment
There was a problem hiding this comment.
LGTM, left some comments.
| @@ -0,0 +1,3 @@ | |||
| # Using {FEATURE} on {BACKEND_NAME} | |||
|
|
|||
| This is a placeholder guide. | |||
There was a problem hiding this comment.
Can we add meta details about the intended use of this section?
|
|
||
| ## Step 1: Environment Setup | ||
|
|
||
| This tutorial is intended to be run from a {SUPPORTED_HOST_OS} and uses Conda for Python environment management. For full setup details and system requirements, see [Getting Started with ExecuTorch](/getting-started). |
42a6b71 to
b577bd8
Compare
67f0493 to
592f698
Compare
| :hidden: | ||
| :caption: {BACKEND} Backend | ||
|
|
||
| backend-troubleshooting |
There was a problem hiding this comment.
np: do you wanna put the troubleshooting section a bit further back in the list?
There was a problem hiding this comment.
I think we went back and forth (literally on the list too :p)
There was a problem hiding this comment.
I updated to put it after partitioner and quant docs, but before arch + internals. Does that work?
| backend-troubleshooting | ||
| backend-partitioner | ||
| backend-quantization | ||
| backend-op-support |
There was a problem hiding this comment.
thinking more, technically op-support can be under partitioner.
So thinking out loud,
- partitioner, quantizer (two entry point, separate pages)
- tutorials (curious why it is not here?)
- internals (can also house reference material facts which can't be part of parishioner or quantizer)
- guide/troubleshooting/gotcha/faq (one page with a title reflecting it is for all these :p)
There was a problem hiding this comment.
That seems reasonable. I've updated to move op support into the partitioner page. I also added tutorials to the TOC in the template - good catch.
|
Under Edge Platforms (iOS, Android, Desktop, C++) etc I don't see XNNPACK anymore on the left nav "Edge->Android->Backends", as well as the overview page https://docs-preview.pytorch.org/pytorch/executorch/14873/android-backends.html (no link to XNNPACK) https://docs-preview.pytorch.org/pytorch/executorch/14873/android-section.html (no XNNPACK on left nav) |
8d763b9 to
1379b98
Compare
|
Now the chevron button doesn't collapse/expand on the left nav https://docs-preview.pytorch.org/pytorch/executorch/14873/android-xnnpack.html?v=10 and the links down below in the page doesn't work |
Summary: Title says it all! Add Vulkan docs in accordance to the format for XNNPACK introduced in #14873.
Summary: Title says it all! Add docs for the Samsung backend based on the template introduced in #14873.
Summary: Title says it all! Add docs for the Samsung backend based on the template introduced in #14873.
Summary: Title says it all! Add Vulkan docs in accordance to the format for XNNPACK introduced in #14873.
Summary: Title says it all! Add docs for the Samsung backend based on the template introduced in #14873.
| backend-op-support | ||
| backend-arch-internals | ||
| tutorials/backend-tutorials | ||
| guides/backend-guides |
There was a problem hiding this comment.
Closing " ``` " missing
Summary: Title says it all! Add Vulkan docs in accordance to the format for XNNPACK introduced in #14873.
Summary: Title says it all! Add Vulkan docs in accordance to the format for XNNPACK introduced in #14873.
796d05c to
3f45145
Compare
3f45145 to
ed22ca6
Compare
|
I've fixed the last couple of links and am merging as is. The issue with expanding the menu for backends under Android/iOS doesn't seem to be resolved when increased the max TOC depth under edge-platform-sections, so we'll need to investigate - we can take as a follow-up. |
|
Thanks @GregoryComer! |
Summary: Title says it all! Add docs for the Samsung backend based on the template introduced in pytorch#14873.
Prototype an updated doc structure for the XNNPACK backend. Extract a common template out under docs/source/backends/tempate/ This PR updates the doc structure as follows. Under the template, the landing page is required, partitioner, quantization, and op support docs are recommended, and the rest are optional. - XNNPACK Backend - Quantization (recommended) - Partitioner APIs (recommended) - Operator Support (optional) - Architecture and Internals (optional) - Tutorials (optional) - Guides (optional)
Updating per template in #14873 Signed-off-by: Robert Kalmar <robert.kalmar@nxp.com>
Updating per template in pytorch#14873 Signed-off-by: Robert Kalmar <robert.kalmar@nxp.com>
Summary: Title says it all! Add docs for the Samsung backend based on the template introduced in pytorch#14873.
### Summary Prototype an updated doc structure for the XNNPACK backend. Extract a common template out under docs/source/backends/tempate/ This PR updates the doc structure as follows. Under the template, the landing page is required, partitioner, quantization, and op support docs are recommended, and the rest are optional. - XNNPACK Backend - Quantization (recommended) - Partitioner APIs (recommended) - Operator Support (optional) - Architecture and Internals (optional) - Tutorials (optional) - Guides (optional)
Updating per template in pytorch#14873 Signed-off-by: Robert Kalmar <robert.kalmar@nxp.com>
Summary
Prototype an updated doc structure for the XNNPACK backend. Extract a common template out under docs/source/backends/tempate/
This PR updates the doc structure as follows. Under the template, the landing page is required, partitioner, quantization, and op support docs are recommended, and the rest are optional.