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

[fbgemm_gpu] Enable building under python -m build #2491

Closed
wants to merge 1 commit into from

Conversation

q10
Copy link
Contributor

@q10 q10 commented Apr 10, 2024

  • Update build scripts to enable building fbgemm_gpu under python -m build instead of python setup.py bdist_wheel

Copy link

netlify bot commented Apr 10, 2024

Deploy Preview for pytorch-fbgemm-docs ready!

Name Link
🔨 Latest commit eff5fe0
🔍 Latest deploy log https://app.netlify.com/sites/pytorch-fbgemm-docs/deploys/6618bf7d2f59960008a33598
😎 Deploy Preview https://deploy-preview-2491--pytorch-fbgemm-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@q10 q10 force-pushed the bm/pip-build branch 3 times, most recently from 19bc675 to e19a076 Compare April 11, 2024 05:00
@q10 q10 changed the title wip [fbgemm_gpu] Enable building under python -m build Apr 11, 2024
@facebook-github-bot
Copy link
Contributor

@q10 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@q10 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@q10 q10 force-pushed the bm/pip-build branch 11 times, most recently from 80edeb0 to 6f670a7 Compare April 11, 2024 22:47
@facebook-github-bot
Copy link
Contributor

@q10 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

- Update build scripts to enable building fbgemm_gpu under `python -m build`
instead of `python setup.py bdist_wheel`
@facebook-github-bot
Copy link
Contributor

@q10 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@q10 merged this pull request in 503cf0e.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants