Skip to content
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

Update method names to support intx and floatx changes #775

Merged
merged 4 commits into from
Aug 28, 2024

Conversation

jainapurva
Copy link
Contributor

@jainapurva jainapurva commented Aug 28, 2024

Updated the following
from_float -> from_hp_to_intx
from_float_static -> from_hp_to_intx_static
from_float_to_floatx -> from_hp_to_floatx

to_affine_quantized -> to_affine_quantized_intx
to_affine_quantized_static -> to_affine_quantized_intx_static

Copy link

pytorch-bot bot commented Aug 28, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/775

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit 6d1d206 with merge base 0916b5b (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@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 28, 2024
@jainapurva jainapurva requested a review from jerryzh168 August 28, 2024 16:01
@jainapurva jainapurva force-pushed the var_name_fixes_aqt branch 2 times, most recently from 4e5d92c to 518fcfe Compare August 28, 2024 16:25
@jerryzh168 jerryzh168 self-requested a review August 28, 2024 17:42
@jerryzh168
Copy link
Contributor

need to resolve some CI errors

@jainapurva jainapurva marked this pull request as ready for review August 28, 2024 21:55
@jainapurva jainapurva merged commit cfabc13 into main Aug 28, 2024
16 checks passed
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