-
Notifications
You must be signed in to change notification settings - Fork 295
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add definition to etrecord_path
to the devtools tutorial
#5458
Conversation
Summary: This diff adds the definition for variable `etrecord_path`. `etrecord_path` was defined in previous steps in the tutorial, but when user gets to this step, they would usually start a new script, so we should define this variable again. Differential Revision: D62921869
🔗 Helpful Links🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/5458
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: ✅ No FailuresAs of commit 2a55757 with merge base 444480b (): This comment was automatically generated by Dr. CI and updates every 15 minutes. |
This pull request was exported from Phabricator. Differential Revision: D62921869 |
This pull request has been merged in b89c52c. |
@pytorchbot cherry-pick --onto release/0.4 -c docs |
Summary: Pull Request resolved: #5458 This diff adds the definition for variable `etrecord_path`. `etrecord_path` was defined in previous steps in the tutorial, but when user gets to this step, they would usually start a new script, so we should define this variable again. Reviewed By: Gasoonjia, dvorjackz Differential Revision: D62921869 fbshipit-source-id: 1d274d5fc76f84f6f943295c142141051fd85cb4 (cherry picked from commit b89c52c)
Cherry picking #5458The cherry pick PR is at #5481 The following tracker issues are updated: Details for Dev Infra teamRaised by workflow job |
Add definition to `etrecord_path` to the devtools tutorial (#5458) Summary: Pull Request resolved: #5458 This diff adds the definition for variable `etrecord_path`. `etrecord_path` was defined in previous steps in the tutorial, but when user gets to this step, they would usually start a new script, so we should define this variable again. Reviewed By: Gasoonjia, dvorjackz Differential Revision: D62921869 fbshipit-source-id: 1d274d5fc76f84f6f943295c142141051fd85cb4 (cherry picked from commit b89c52c) Co-authored-by: Olivia Liu <olivialpx@meta.com>
Summary: This diff adds the definition for variable
etrecord_path
.etrecord_path
was defined in previous steps in the tutorial, but when user gets to this step, they would usually start a new script, so we should define this variable again.Differential Revision: D62921869