-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CLN: ASV io_bench, parser_vb #18815
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: ASV io_bench, parser_vb #18815
Conversation
Codecov Report
@@ Coverage Diff @@
## master #18815 +/- ##
==========================================
- Coverage 91.64% 91.62% -0.02%
==========================================
Files 154 154
Lines 51408 51408
==========================================
- Hits 47113 47104 -9
- Misses 4295 4304 +9
Continue to review full report at Codecov.
|
+1 on this, does asv supported nested benchmarks, e.g. can we create a |
7b6a72c
to
e8eda6a
Compare
Yup. Created a new
|
thanks @mroeschke ideally would consolidate the rest of the io benchmarks in io/ |
There are a lot of io benchmarks scattered among
hdfstore_bench.py
,io_bench.py
,parser_vb.py
, andpacker.py
. I think it would be a lot cleaner if each io method had it's own file (likeio_sql.py
which already exists)This PR creates
io_csv.py
andio_json.py
that consolidates the csv and json benchmarks that exist. Benchmarks were flake8'd,param
'd, and simplified where available: