-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
test(python): Update benchmark tests #15825
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #15825 +/- ##
==========================================
- Coverage 80.38% 80.38% -0.01%
==========================================
Files 1263 1263
Lines 165346 165368 +22
==========================================
+ Hits 132918 132926 +8
- Misses 32428 32442 +14 ☔ View full report in Codecov by Sentry. |
CodSpeed Performance ReportMerging #15825 will improve performances by ×560Comparing Summary
Benchmarks breakdown
|
Is 10_000 or 50_000 still acceptable? I feel 1000 is too nano for Polars. |
I think that would be fine. I updated to 10_000 rows, let's see how long each benchmark test takes now. |
Changes
100010_000 rows.