Skip to content

SDK example runner changes to suppport intermediate tensor logging #1307

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

Closed
wants to merge 1 commit into from

Conversation

tarun292
Copy link
Contributor

Summary: This diff adds support to the sdk_example_runner to take in command line parameters to enable intermediate/program output logging to etdump. Along with that we take in additional parameters for debug_buffer path and debug_buffer size. The debug_buffer is where the tensor and scalar data is stored (just the raw data not the metadata).

Reviewed By: Jack-Khuu

Differential Revision: D51599690

Copy link

pytorch-bot bot commented Nov 29, 2023

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit 64a6c3c with merge base 388490c (image):

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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

@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Nov 29, 2023
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51599690

facebook-github-bot pushed a commit that referenced this pull request Nov 29, 2023
…1307)

Summary:

This diff adds support to the `sdk_example_runner` to take in command line parameters to enable intermediate/program output logging to etdump. Along with that we take in additional parameters for debug_buffer path and debug_buffer size. The debug_buffer is where the tensor and scalar data is stored (just the raw data not the metadata).

Reviewed By: Jack-Khuu

Differential Revision: D51599690
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51599690

tarun292 added a commit that referenced this pull request Nov 29, 2023
…1307)

Summary:

This diff adds support to the `sdk_example_runner` to take in command line parameters to enable intermediate/program output logging to etdump. Along with that we take in additional parameters for debug_buffer path and debug_buffer size. The debug_buffer is where the tensor and scalar data is stored (just the raw data not the metadata).

Reviewed By: Jack-Khuu

Differential Revision: D51599690
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51599690

…1307)

Summary:

This diff adds support to the `sdk_example_runner` to take in command line parameters to enable intermediate/program output logging to etdump. Along with that we take in additional parameters for debug_buffer path and debug_buffer size. The debug_buffer is where the tensor and scalar data is stored (just the raw data not the metadata).

Reviewed By: Jack-Khuu

Differential Revision: D51599690
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D51599690

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in cb2ff0f.

@facebook-github-bot facebook-github-bot deleted the export-D51599690 branch December 3, 2023 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants