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 quantization doc: add x86 backend as default backend of server inference #86900

Closed

Conversation

XiaobingSuper
Copy link
Collaborator

#83888 introduces x86 backedn to unify the x86 CPUs quantization, this PR updates the quantization tutorial by setting x86 backend as the default backend on x86 cpus.

@pytorch-bot
Copy link

pytorch-bot bot commented Oct 13, 2022

🔗 Helpful Links

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

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

✅ No Failures

As of commit bc4361a:
💚 Looks good so far! There are no failures yet. 💚

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

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Oct 13, 2022
@XiaobingSuper XiaobingSuper added the module: docs Related to our documentation, both in docs/ and docblocks label Oct 13, 2022
Copy link
Contributor

@malfet malfet left a comment

Choose a reason for hiding this comment

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

Hmm, wasn't it reverted in #86631 in trunk and in #86726 in release/1.13, so the doc update would be misleading to say the least

@jgong5
Copy link
Collaborator

jgong5 commented Oct 14, 2022

Hmm, wasn't it reverted in #86631 in trunk and in #86726 in release/1.13, so the doc update would be misleading to say the least

@malfet We are closely working with @jerryzh168 to address the test failure and a PR fix #86876 is under verification. We submitted this doc update to catch the deadline of doc/tutorial update, just in case the failure can be fixed for 1.13...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request module: docs Related to our documentation, both in docs/ and docblocks open source
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants