Skip to content

Make WeightOnlyInt8QuantLinear more compatible with torch.compile #19

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cpuhrsch
Copy link
Contributor

@cpuhrsch cpuhrsch commented Dec 3, 2023

No description provided.

@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 Dec 3, 2023
@HDCharles
Copy link
Contributor

may be good to align the forward method with the tensor subclass as well.

)
new_mod.register_buffer("w_int8", w_int8.t())
new_mod.scales = torch.nn.Parameter(scales)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably want requires_grad = False

jerryzh168 pushed a commit that referenced this pull request Sep 18, 2024
* Bring `torch.compile` to `quant_block_v2_`. (#18)

Signed-off-by: yiliu30 <[email protected]>

* Add `AO_USE_DETERMINISTIC_ALGORITHMS` for reproducing results (#19)

Signed-off-by: yiliu30 <[email protected]>

* Add `gradient_accumulate_steps` and update results (#20)

Signed-off-by: yiliu30 <[email protected]>

* update the readme

Signed-off-by: yiliu30 <[email protected]>

* udpate

Signed-off-by: yiliu30 <[email protected]>

* update the desc

Signed-off-by: yiliu30 <[email protected]>

* rename `train_bs` to `batch_size`

Signed-off-by: yiliu30 <[email protected]>

* update the eval

Signed-off-by: yiliu30 <[email protected]>

* update

Signed-off-by: yiliu30 <[email protected]>

---------

Signed-off-by: yiliu30 <[email protected]>
jainapurva pushed a commit that referenced this pull request Sep 22, 2024
* Bring `torch.compile` to `quant_block_v2_`. (#18)

Signed-off-by: yiliu30 <[email protected]>

* Add `AO_USE_DETERMINISTIC_ALGORITHMS` for reproducing results (#19)

Signed-off-by: yiliu30 <[email protected]>

* Add `gradient_accumulate_steps` and update results (#20)

Signed-off-by: yiliu30 <[email protected]>

* update the readme

Signed-off-by: yiliu30 <[email protected]>

* udpate

Signed-off-by: yiliu30 <[email protected]>

* update the desc

Signed-off-by: yiliu30 <[email protected]>

* rename `train_bs` to `batch_size`

Signed-off-by: yiliu30 <[email protected]>

* update the eval

Signed-off-by: yiliu30 <[email protected]>

* update

Signed-off-by: yiliu30 <[email protected]>

---------

Signed-off-by: yiliu30 <[email protected]>
jainapurva pushed a commit that referenced this pull request Sep 23, 2024
* Bring `torch.compile` to `quant_block_v2_`. (#18)

Signed-off-by: yiliu30 <[email protected]>

* Add `AO_USE_DETERMINISTIC_ALGORITHMS` for reproducing results (#19)

Signed-off-by: yiliu30 <[email protected]>

* Add `gradient_accumulate_steps` and update results (#20)

Signed-off-by: yiliu30 <[email protected]>

* update the readme

Signed-off-by: yiliu30 <[email protected]>

* udpate

Signed-off-by: yiliu30 <[email protected]>

* update the desc

Signed-off-by: yiliu30 <[email protected]>

* rename `train_bs` to `batch_size`

Signed-off-by: yiliu30 <[email protected]>

* update the eval

Signed-off-by: yiliu30 <[email protected]>

* update

Signed-off-by: yiliu30 <[email protected]>

---------

Signed-off-by: yiliu30 <[email protected]>
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants