Update docs on Module new APIs. #5952
Conversation
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/5952
Note: Links to docs will display an error until the docs builds have been completed. ✅ No FailuresAs of commit be1eb15 with merge base f005dd5 ( This comment was automatically generated by Dr. CI and updates every 15 minutes. |
facebook-github-bot
added
the
CLA Signed
|
This pull request was exported from Phabricator. Differential Revision: D64005568 |
docs/source/extension-module.md
Outdated
|
|
||
| #### Setting Outputs | ||
|
|
||
| Currently, only tensor outputs are supported for setting, and they must not be memory-planned at model export time. Memory-planned tensors are preallocated during model export and cannot be replaced. |
There was a problem hiding this comment.
This would be more clear:
"Only outputs of type Tensor can be set at runtime, and ..."
When I first read the current version I interpreted it as "Only outputs of tensors" (which doesn't really mean anything) rather than "only outputs that are tensors".
Summary: Pull Request resolved: #5952 Reviewed By: kirklandsign Differential Revision: D64005568
|
This pull request was exported from Phabricator. Differential Revision: D64005568 |
|
@pytorchbot cherry-pick --onto release/0.4 -c docs |
|
This pull request has been merged in 0424eef. |
|
@pytorchbot cherry-pick --onto release/0.4 -c docs |
Cherry picking #5952The cherry pick PR is at #5968 The following tracker issues are updated: Details for Dev Infra teamRaised by workflow job |
5 participants
Differential Revision: D64005568