-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
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
CLN: clean benchmarks to get them running #16025
CLN: clean benchmarks to get them running #16025
Conversation
this is great not sure if docs need any updates |
Almost forgot, there was one remaining error related to hdf5:
Benchmarks are here: https://github.com/pandas-dev/pandas/blob/master/asv_bench/benchmarks/hdfstore_bench.py#L87-L93 |
rename table variables
add a commit to use the correct syntax. |
Codecov Report
@@ Coverage Diff @@
## master #16025 +/- ##
==========================================
- Coverage 90.99% 90.99% -0.01%
==========================================
Files 153 153
Lines 50470 50469 -1
==========================================
- Hits 45927 45926 -1
Misses 4543 4543
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #16025 +/- ##
==========================================
- Coverage 90.99% 90.99% -0.01%
==========================================
Files 153 153
Lines 50470 50469 -1
==========================================
- Hits 45927 45926 -1
Misses 4543 4543
Continue to review full report at Codecov.
|
Thanks! (BTW, you should link your two sigma address to your github account, it is not yet recognizing your commits) |
Changed to 3.6 as well! |
where do you see a problem? side issue - I have a couple more small reorgs (indexes, tools) coming |
If you look at the commit you added in this PR, it doesn't have your icon, and if you look at the commit: 7e4b9dc, the question mark next to the author gives some more explanation |
e-mail address fixed thanks! |
Changes to be compatible with latest master / py3.