Skip to content

Pybindings Module is more flexible with non_const set ups #109

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

JacobSzwejbka
Copy link
Contributor

Summary: Made the module handle models with more then one non_const buffer level, and made it allocate the needed memory dynamically rather then relying on static constants which historically have needed to be updated a ton.

Differential Revision: D48583639

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

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

JacobSzwejbka and others added 3 commits August 23, 2023 11:43
Summary:
Pull Request resolved: pytorch/executorch#98

title

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

fbshipit-source-id: 6ad2d696635737043ae3ff6097601597f2c98478
Summary:
Pull Request resolved: pytorch/executorch#97

unused so deleting

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

fbshipit-source-id: 761270610694853687dbe5527aa9c014bff7d1c8
Summary:
Pull Request resolved: pytorch/executorch#109

Made the module handle models with more then one non_const buffer level, and made it allocate the needed memory dynamically rather then relying on static constants which historically have needed to be updated a ton.

Differential Revision: D48583639

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in f4ed04f.

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