-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Update cibuildwheel #23024
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 cibuildwheel #23024
Conversation
Codecov Report
@@ Coverage Diff @@
## master #23024 +/- ##
==========================================
- Coverage 73.71% 73.69% -0.02%
==========================================
Files 714 714
Lines 95240 95240
==========================================
- Hits 70203 70190 -13
- Misses 23740 23753 +13
Partials 1297 1297
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
| from the inference.""" | ||
| PredictionResult.example.__doc__ = """The input example.""" | ||
| PredictionResult.inference.__doc__ = """Results for the inference on the model | ||
| PredictionResult.inference.__doc__ = """Results fo r the inference on the model |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will remove this once this PR gets LGTM
|
Assigning reviewers. If you would like to opt out of this review, comment R: @TheNeuralBit for label python. Available commands:
The PR bot will only process comments in the main thread (not review comments). |
|
stop reviewer notifications |
tvalentyn
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please file an issue to add support for new wheel architecture (and add a TODO), or increase the timeout. Thanks!
|
Stopping reviewer notifications for this pull request: requested by reviewer |
This reverts commit 221e3fb.
|
@tvalentyn added a TODO with issue link. PTAL |
Update
cibuildwheelto a latest version and exclude building themuslinuxwheelsThank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
R: @username).addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, commentfixes #<ISSUE NUMBER>instead.CHANGES.mdwith noteworthy changes.See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)
See CI.md for more information about GitHub Actions CI.