Skip to content

[executorch] Print executor_runner outputs using EValue's new operator<<() #481

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 3 commits into from

Conversation

dbort
Copy link
Contributor

@dbort dbort commented Sep 25, 2023

Stack from ghstack (oldest at bottom):

D49574853 added a smart operator<<() for EValue. Instead of assuming that all outputs are fb32 tensors, use the new printer to print each output EValue.

By default it will only print the first and last three entries in a list, so extend it to print the first and last 100 entries.

Differential Revision: D49607604

…r<<()

D49574853 added a smart `operator<<()` for `EValue`. Instead of assuming that all outputs are fb32 tensors, use the new printer to print each output `EValue`.

By default it will only print the first and last three entries in a list, so extend it to print the first and last 100 entries.

Differential Revision: [D49607604](https://our.internmc.facebook.com/intern/diff/D49607604/)

[ghstack-poisoned]
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 25, 2023
dbort added a commit that referenced this pull request Sep 25, 2023
…r<<()

D49574853 added a smart `operator<<()` for `EValue`. Instead of assuming that all outputs are fb32 tensors, use the new printer to print each output `EValue`.

By default it will only print the first and last three entries in a list, so extend it to print the first and last 100 entries.

Differential Revision: [D49607604](https://our.internmc.facebook.com/intern/diff/D49607604/)

ghstack-source-id: 201836987
Pull Request resolved: #481
…new operator<<()"

D49574853 added a smart `operator<<()` for `EValue`. Instead of assuming that all outputs are fb32 tensors, use the new printer to print each output `EValue`.

By default it will only print the first and last three entries in a list, so extend it to print the first and last 100 entries.

Differential Revision: [D49607604](https://our.internmc.facebook.com/intern/diff/D49607604/)

[ghstack-poisoned]
dbort added a commit that referenced this pull request Sep 25, 2023
…r<<()

Pull Request resolved: #481

D49574853 added a smart `operator<<()` for `EValue`. Instead of assuming that all outputs are fb32 tensors, use the new printer to print each output `EValue`.

By default it will only print the first and last three entries in a list, so extend it to print the first and last 100 entries.
ghstack-source-id: 201845602
@exported-using-ghexport

Differential Revision: [D49607604](https://our.internmc.facebook.com/intern/diff/D49607604/)
…new operator<<()"

D49574853 added a smart `operator<<()` for `EValue`. Instead of assuming that all outputs are fb32 tensors, use the new printer to print each output `EValue`.

By default it will only print the first and last three entries in a list, so extend it to print the first and last 100 entries.

Differential Revision: [D49607604](https://our.internmc.facebook.com/intern/diff/D49607604/)

[ghstack-poisoned]
dbort added a commit that referenced this pull request Sep 25, 2023
…r<<()

Pull Request resolved: #481

D49574853 added a smart `operator<<()` for `EValue`. Instead of assuming that all outputs are fb32 tensors, use the new printer to print each output `EValue`.

By default it will only print the first and last three entries in a list, so extend it to print the first and last 100 entries.
ghstack-source-id: 201868940
@exported-using-ghexport

Differential Revision: [D49607604](https://our.internmc.facebook.com/intern/diff/D49607604/)
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 13bac84.

@facebook-github-bot facebook-github-bot deleted the gh/dbort/21/head branch September 29, 2023 14:21
Gasoonjia pushed a commit that referenced this pull request Jul 30, 2024
* Add build_native.sh and add README.md

Summary:

Added a script to build C++ runner for ET and AOTI. Updated README.md to
ask users to run it.

Made some improvement on building speed, by reducing duplicate build
command. Now we can rely on `install_requirements.sh` to install all of
the C++ libraries needed for runner.

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

* Revert custom ops change

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

* Add build_native.sh to CI job

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:

* Add README for building native runner for aoti

Summary:

Test Plan:

Reviewers:

Subscribers:

Tasks:

Tags:
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. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants