Skip to content

[executorch] Clean up MemoryManager API #402

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

dbort
Copy link
Contributor

@dbort dbort commented Sep 19, 2023

Stack from ghstack (oldest at bottom):

  • Remove const allocator, which we don't use
  • Don't make users create empty allocators for entries they don't use
  • Pick more-descriptive names

Differential Revision: D49389211

- Remove const allocator, which we don't use
- Don't make users create empty allocators for entries they don't use
- Pick more-descriptive names

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

[ghstack-poisoned]
- Remove const allocator, which we don't use
- Don't make users create empty allocators for entries they don't use
- Pick more-descriptive names

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

[ghstack-poisoned]
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 6944c45.

@facebook-github-bot facebook-github-bot deleted the gh/dbort/15/head branch September 23, 2023 14:20
Gasoonjia pushed a commit that referenced this pull request Jul 30, 2024
* ET or AOTI backend logic

* use args, not builder_args

* typo

* typo

* typo
Gasoonjia pushed a commit that referenced this pull request Jul 30, 2024
This reverts commit 052fb1a3792123e291fb5f47084644913c8ecba7 to  fix [test-tinystories-executorch](https://github.com/pytorch/torchchat/actions/runs/8802811768/job/24159454773?pr=411#logs)
Gasoonjia pushed a commit that referenced this pull request Jul 30, 2024
* Revert "Revert "Embedding quantization per backend (#402)" (#411)"

This reverts commit 8b35acdff4fded779799ab8a419e55f885dd8918.

* 4b and 8b embedding table quantization

* minor changes

* remove extra et workflow
Gasoonjia pushed a commit that referenced this pull request Jul 30, 2024
* Revert "Revert "Embedding quantization per backend (#402)" (#411)"

This reverts commit 8b35acdff4fded779799ab8a419e55f885dd8918.

* merge GGUF tests into pull.yml
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