Skip to content

Improvements to gen_sample_etrecord #1073

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

Conversation

vmpuri
Copy link

@vmpuri vmpuri commented Oct 23, 2023

Summary:
Various improvements to the gen_sample_etrecord script.

  • Enable users to generate an etrecord from any of the sample models
  • Output files to /tmp instead of the working directory, provide a CLI option for this
  • Clean up imports and add type hints

Differential Revision: D50579104

@netlify
Copy link

netlify bot commented Oct 23, 2023

Deploy Preview for resplendent-gnome-14e531 canceled.

Name Link
🔨 Latest commit 62e5a46
🔍 Latest deploy log https://app.netlify.com/sites/resplendent-gnome-14e531/deploys/653bdc22e684340008768d9a

@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 Oct 23, 2023
@facebook-github-bot
Copy link
Contributor

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

1 similar comment
@facebook-github-bot
Copy link
Contributor

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

vmpuri pushed a commit to vmpuri/executorch-1 that referenced this pull request Oct 27, 2023
Summary:
Pull Request resolved: pytorch#1073

Various improvements to the gen_sample_etrecord script.
- Enable users to generate an etrecord from any of the sample models
- Output files to /tmp instead of the working directory, provide a CLI option for this
- Clean up imports and add type hints

Differential Revision: D50579104

fbshipit-source-id: 40ee8297dba3bbe6f0bada5e819863b08e77862d
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 27, 2023

🔗 Helpful Links

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

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

✅ No Failures

As of commit 62e5a46 with merge base 251801e (image):
💚 Looks good so far! There are no failures yet. 💚

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

@facebook-github-bot
Copy link
Contributor

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

vmpuri pushed a commit to vmpuri/executorch-1 that referenced this pull request Oct 27, 2023
Summary:
Pull Request resolved: pytorch#1073

Various improvements to the gen_sample_etrecord script.
- Enable users to generate an etrecord from any of the sample models
- Output files to /tmp instead of the working directory, provide a CLI option for this
- Clean up imports and add type hints

Reviewed By: Jack-Khuu

Differential Revision: D50579104

fbshipit-source-id: 5ac5ebe981d37aaa94835f9faf71a9f883f55f1e
@facebook-github-bot
Copy link
Contributor

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

vmpuri pushed a commit to vmpuri/executorch-1 that referenced this pull request Oct 27, 2023
Summary:
Pull Request resolved: pytorch#1073

Various improvements to the gen_sample_etrecord script.
- Enable users to generate an etrecord from any of the sample models
- Output files to /tmp instead of the working directory, provide a CLI option for this
- Clean up imports and add type hints

Reviewed By: Jack-Khuu

Differential Revision: D50579104

fbshipit-source-id: a574a4723a0a2345336d61c1d7a62d669c2e3953
@facebook-github-bot
Copy link
Contributor

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

vmpuri pushed a commit to vmpuri/executorch-1 that referenced this pull request Oct 27, 2023
Summary:
Pull Request resolved: pytorch#1073

Various improvements to the gen_sample_etrecord script.
- Enable users to generate an etrecord from any of the sample models
- Output files to /tmp instead of the working directory, provide a CLI option for this
- Clean up imports and add type hints

Reviewed By: Jack-Khuu

Differential Revision: D50579104

fbshipit-source-id: d04c2f9b81352cfb8bc3cbadc5e9c82cc9200643
Varun Puri and others added 2 commits October 27, 2023 08:48
Summary: This script is useful to generate a sample ETRecord from the example models provided in the repo. This should help users explore the features of the ETrecord without needing to first export a .pte model.

Differential Revision: https://internalfb.com/D50521582

fbshipit-source-id: abf3702f1c4acc83844de7806b441b776927f01e
Summary:
Pull Request resolved: pytorch#1073

Various improvements to the gen_sample_etrecord script.
- Enable users to generate an etrecord from any of the sample models
- Output files to /tmp instead of the working directory, provide a CLI option for this
- Clean up imports and add type hints

Reviewed By: Jack-Khuu

Differential Revision: D50579104

fbshipit-source-id: 574f612b0c8a880ea96dda99431198c393d9bb2f
@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 7814dd7.

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