-
Notifications
You must be signed in to change notification settings - Fork 8
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
Build wheel on gcc 9 (like pytorch) and add sanity check for wheel #76
Conversation
@NicolasHug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@NicolasHug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@NicolasHug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@NicolasHug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@NicolasHug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@NicolasHug has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@NicolasHug merged this pull request in cc0c71d. |
This PR ensures that we build our wheel on gcc 9 (like torch) and adds a sanity check for wheel compatibility. This should prevent a common problem that @ahmadsharif1 and I have often observed when building locally. Please see comments in the code for a full explanation of the issue we're addressing.