Repository navigation
[ENH] Make Parquet downloads configurable - #1752
Jayant-kernel wants to merge 3 commits into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1752 +/- ##
==========================================
- Coverage 53.99% 53.77% -0.22%
==========================================
Files 63 63
Lines 5230 5229 -1
==========================================
- Hits 2824 2812 -12
- Misses 2406 2417 +11 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
There is one failed test that I would like to looked deeper into and verify that it is not caused directly by this, in which case I can directly merge this. |
|
@L-Agbaria |
|
@PGijsbers |
74d4613 to
3d73245
Compare
L-Agbaria
left a comment
There was a problem hiding this comment.
This feature will be added until parquet files become the default, after which this will resolved to always download parquet.
|
Glad this works for the transition period |
Metadata
Details
This adds a persistent
skip_parquetconfiguration option for dataset downloads. When enabled, datasets use the ARFF download path even when a Parquet URL is available.The existing
OPENML_SKIP_PARQUETenvironment variable remains supported. Both dataset download paths now use the same configuration check so the behaviour is consistent.Tests cover loading
skip_parquetfrom the config file and confirm that the dataset download path skips Parquet and falls back to ARFF when the option is enabled.