Skip to content

update set_tensor_data to use MethodMeta and set_input #524

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

JacobSzwejbka
Copy link
Contributor

Summary:
Switch the util used in all of our examples to use "set_input" instead of get_input, set_data.

Theres still more work to be done on this util like make it follow our style guide, put it in a more specific file, use RAII but thatll be after MVP.

Differential Revision: D49747239

@netlify
Copy link

netlify bot commented Sep 28, 2023

Deploy Preview for resplendent-gnome-14e531 canceled.

Name Link
🔨 Latest commit ca45bbf
🔍 Latest deploy log https://app.netlify.com/sites/resplendent-gnome-14e531/deploys/651c6fab793a41000870e1fd

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

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

@facebook-github-bot
Copy link
Contributor

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

@facebook-github-bot
Copy link
Contributor

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

JacobSzwejbka and others added 2 commits October 3, 2023 12:44
Summary:
Pull Request resolved: pytorch/executorch#523

None of these should exist in the public API, but we are out of time before MVP so just deprecating. Added a way to get a MethodMeta from method.

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

fbshipit-source-id: 0b2367641d1d2174b33a7d80246161f9f54f3a8f
Summary:
Pull Request resolved: pytorch/executorch#524

Switch the util used in all of our examples to use "set_input" instead of get_input, set_data.

Theres still more work to be done on this util like make it follow our style guide, put it in a more specific file, use RAII but thatll be after MVP.

Reviewed By: dbort

Differential Revision: D49747239

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 2a02032.

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