Skip to content

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